(Trivial) Rename TxName

[kevinrr888]

Is your feature request related to a problem? Please describe.
Prior to #5218, a String was passed around and stored representing the type of fate operation being performed. This was changed to be a concrete type. TxName and similar naming are still used throughout the FATE code, which isn't as representative of what is being passed/stored now.

Describe the solution you'd like
All mentions of TxName or similar should be renamed to something more appropriate like fateOp, txOp, txOperation, etc.

[richochetclementine1315]

Hi @kevinrr888 myself Mrinmoy. First of all I want to ask is this issue already assigned to someone??
and secondly...Can you pls explain a bit about this issue??

THANKS :)

[kevinrr888]

Hi @richochetclementine1315
#5218 introduced a new type FateOperation as a replacement to passing around a String. It represents the type of fate operation being performed. For example, if a table is created, this would be a TABLE_CREATE fate operation. TxName, txName, or similar were used throughout the code to represent this previously String type. This is no longer the best way to represent what is being stored or passed around now. fateOp may be the best way to represent this now. Grepping for txname (case insensitive) would be a good starting place to complete this ticket. More info about fate can be found here if curious https://accumulo.apache.org/docs/2.x/administration/fate.

Let me know if you would like me to assign this ticket to you.

[richochetclementine1315]

yes...I wanna work upon this issue @kevinrr888 :)

[richochetclementine1315]

@kevinrr888 I just did a PR.... Kindly take a look... please feel free to tell if some other changes are required. Thanks!! :)

[richochetclementine1315]

hi @kevinrr888... I've made the changes. Please take a look THANKS :)