milleruntime commented on pull request #2215: URL: https://github.com/apache/accumulo/pull/2215#issuecomment-1059327344
These changes look pretty good and I think this would be nice to get into 2.1. @Manno15 what kind of testing were you able to do? My only concern is leaking the FATE types in `TransactionStatus`. We do blind casting of the serialized FATE repos on the server side so I think it might be possible to return some of the types not in the API. -- 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]
