brianloss commented on a change in pull request #1907:
URL: https://github.com/apache/accumulo/pull/1907#discussion_r570441340



##########
File path: core/src/main/thrift/manager.thrift
##########
@@ -213,7 +213,7 @@ service FateService {
 
 }
 
-service MasterClientService extends FateService {

Review comment:
       I spun up a cluster from a tarball built from the main branch, and then 
ran the accumulo shell from a 2.0.1 install (but client properties point to my 
2.1 snapshot instance), and while some things worked in the shell, scanning a 
table hung indefinitely (but worked from the 2.1 shell), so I'd say client 
compatibility was already broken. Given that, it would seem the best approach 
is just to document that fact in the release notes.




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

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


Reply via email to