DomGarguilo commented on PR #4789:
URL: https://github.com/apache/accumulo/pull/4789#issuecomment-2278232253

   > I'm against this change. See my notes about the confusing semantics in 
[#4785 
(comment)](https://github.com/apache/accumulo/pull/4785#discussion_r1708177809)
   > 
   > I would prefer to see this reverted in favor of a more clear API. I can 
make a more concrete suggestion for what that API would look like once I catch 
up with these changes. I'm a little behind due to being on vacation and 
currently being sick.
   
   I am open to considering reverting this change once you finalize your 
suggestion. For now since this is already merged and another PR is waiting on 
this change, ill merge it up and we can fix things later in a new PR if needed.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to