You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/08/02 22:31:47 UTC

[GitHub] [accumulo] cshannon commented on pull request #2842: Renaming ZooStore get/set property methods to get/set nodeData

cshannon commented on PR #2842:
URL: https://github.com/apache/accumulo/pull/2842#issuecomment-1203277106

   @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.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org