andrewglowacki commented on issue #1584: Moved ThriftClientHandler out of TabletServer for #1581 URL: https://github.com/apache/accumulo/pull/1584#issuecomment-612949478 @keith-turner sorry, what changes do you mean? The force push didn't add/remove anything from the original commit, it just removed the commits that I accidently pulled in when I rebased master. This is what happened: 1. Original commit: 8d99cdc 2. Reviewed by @ctubbsii 3. I rebased master which pulled in and modified the next 4 commits 4. Pushed this to my remote branch before I realized the 4 commits were modified by the rebase 5. Reset the head to 8d99cdc 6. Force push the branch to remove the 4 commits 7. Committed and push my changes from the review in 1bf3ba5
---------------------------------------------------------------- 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] With regards, Apache Git Services
