[GitHub] [iceberg] XN137 commented on pull request #6986: Build: Upgrade netty-buffer to 4.1.89.Final

["XN137 (via GitHub)" <gi...@apache.org>]

XN137 commented on PR #6986:
URL: https://github.com/apache/iceberg/pull/6986#issuecomment-1454661186

   this is still a draft so not ready for review
   
   that being said, the current version `4.1.68.Final` is a version downgrade since the recent arrow upgrade... so i am guessing the current version pin only continues to work because neither spark nor arrow are using APIs that are only available in their newer netty versions.
   
   also not sure why we pin the version ourselves when we want to keep it in sync with one (both?) of the other dependencies, when we could let one of those dependencies pull in "their version"
   
   did we ever hit netty compatibility issues that are not covered by iceberg CI ?
   
   


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