You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tez.apache.org by GitBox <gi...@apache.org> on 2021/02/08 20:59:26 UTC
[GitHub] [tez] belugabehr opened a new pull request #106: TEZ-4285: Drop Commons Codec Dependency
belugabehr opened a new pull request #106:
URL: https://github.com/apache/tez/pull/106
----------------------------------------------------------------
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] [tez] belugabehr closed pull request #106: TEZ-4285: Drop Commons Codec Dependency
Posted by GitBox <gi...@apache.org>.
belugabehr closed pull request #106:
URL: https://github.com/apache/tez/pull/106
--
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] [tez] jteagles commented on pull request #106: TEZ-4285: Drop Commons Codec Dependency
Posted by GitBox <gi...@apache.org>.
jteagles commented on pull request #106:
URL: https://github.com/apache/tez/pull/106#issuecomment-781486778
@belugabehr, this change may be problematic. In my experience, Base64 encoding is not a standard and cannot guarantee that two implementations will encode the same way. When strings are encoded and used in multiple components, then will lead to breakages. In this case, I believe there may be a hash that is exchanged with the shuffle system.
----------------------------------------------------------------
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