apache
Renaming ZooStore get/set property methods to get/set nodeData
Also using a new enum type for the node names instead of String values.
This closes #2841
@EdColeman - Here is my first attempt at renaming the methods and using an enum. Per our discussion offline I renamed the "debug" property to "target" as it's a better description of what it is. Obviously any of those names can be changed or tweaked based on what people think. I didn't add any new tests as I figure the existing Fate tests should catch any issues with the ZooStore and refactoring.
@EdColeman - Thanks for the quick review, I'll look at making these changes when I get some time later this week either tomorrow or this weekend.
@EdColeman and @dlmarion - I updated my PR with a new commit to address the comments. I renamed the enum to TxInfo and the methods. Also another change is the enum name is used as is now when storing data into ZK which is all caps instead of lower case (this can be changed of course but seems ok to me).
Looks good but there were a few references to old name (property) still hanging around in the javadoc.
Thanks, I can fix those tomorrow morning and then we can hopefully get this merged in.
@milleruntime - Ok I added a new commit to fix the Javadoc and enum. (I also rebased off main to make sure no conflicts or issues). This should be ready to merged into main I think.
