You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2020/10/19 23:11:28 UTC

[GitHub] [iceberg] kbendick commented on pull request #1636: WIP - Draft to remove NarrowingCompoundAssignment warnings

kbendick commented on pull request #1636:
URL: https://github.com/apache/iceberg/pull/1636#issuecomment-712490984


   cc @rdblue was @RussellSpitzer if you could take a look at my draft approach to removing a good handful of `NarrowingCompoundAssignment` warnings.
   
   Specifically, I'm not 100% sure that there isn't still an issue with `cv.childCount` still being overflown. But that depends on whether or not `output` is constantly reused. Your input would be appreciated before I fix the rest of them (one way or another).


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



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