You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "danielcweeks (via GitHub)" <gi...@apache.org> on 2023/04/03 18:32:50 UTC

[GitHub] [iceberg] danielcweeks opened a new pull request, #7274: Use 6 bytes for path encoding buffer to avoid filler characters

danielcweeks opened a new pull request, #7274:
URL: https://github.com/apache/iceberg/pull/7274

   This avoids using `=` characters as filler in path locations so that they are not used as part of the hash prefix.
   
   Before
   ```
   jshell> enc.encodeToString(new byte[4])
   $8 ==> "AAAAAA=="
   ```
   
   After
   ```
   jshell> enc.encodeToString(new byte[6])
   $9 ==> "AAAAAAAA"
   ```


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] danielcweeks closed pull request #7274: Use 6 bytes for path encoding buffer to avoid filler characters

Posted by "danielcweeks (via GitHub)" <gi...@apache.org>.
danielcweeks closed pull request #7274: Use 6 bytes for path encoding buffer to avoid filler characters
URL: https://github.com/apache/iceberg/pull/7274


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] danielcweeks commented on pull request #7274: Use 6 bytes for path encoding buffer to avoid filler characters

Posted by "danielcweeks (via GitHub)" <gi...@apache.org>.
danielcweeks commented on PR #7274:
URL: https://github.com/apache/iceberg/pull/7274#issuecomment-1494792244

   (cc @singhpk234) small follow up to avoid `=` chars


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org