[GitHub] [accumulo] ctubbsii opened a new pull request #2176: Fix row encoding/decoding in LogFileKey

[GitBox <gi...@apache.org>]

ctubbsii opened a new pull request #2176:
URL: https://github.com/apache/accumulo/pull/2176


   Fix bitwise operations to encode and decode the tabletId and sequence
   number to/from a row
   
   This fixes #2173


-- 
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:
users@infra.apache.org

[GitHub] [accumulo] milleruntime commented on pull request #2176: Fix row encoding/decoding in LogFileKey

[GitBox <gi...@apache.org>]

milleruntime commented on pull request #2176:
URL: https://github.com/apache/accumulo/pull/2176#issuecomment-866798253


   @ctubbsii I opened https://github.com/apache/accumulo/pull/2177 with the fixes I had for this from yesterday before seeing this PR. I think your changes to the byte decoding are better but I have comments and tests I added as well, which I would like to merge. I am going to merge your changes so I can resolve them with mine.


-- 
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:
users@infra.apache.org

[GitHub] [accumulo] milleruntime merged pull request #2176: Fix row encoding/decoding in LogFileKey

[GitBox <gi...@apache.org>]

milleruntime merged pull request #2176:
URL: https://github.com/apache/accumulo/pull/2176


   


-- 
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:
users@infra.apache.org