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 2022/08/15 09:05:26 UTC

[GitHub] [iceberg] XN137 opened a new pull request, #5529: Build: enforce LambdaMethodReference check at compile-time

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

   Follow-up to https://github.com/apache/iceberg/pull/5476 where the last violation was removed
   
   the idea is to reduce review effort by enforcing this preference through a compile-time check instead of an ignorable warning


-- 
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] rdblue merged pull request #5529: Build: enforce LambdaMethodReference check at compile-time

Posted by GitBox <gi...@apache.org>.
rdblue merged PR #5529:
URL: https://github.com/apache/iceberg/pull/5529


-- 
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] rdblue commented on pull request #5529: Build: enforce LambdaMethodReference check at compile-time

Posted by GitBox <gi...@apache.org>.
rdblue commented on PR #5529:
URL: https://github.com/apache/iceberg/pull/5529#issuecomment-1221631140

   This seems like a good idea to me.


-- 
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] nastra commented on pull request #5529: Build: enforce LambdaMethodReference check at compile-time

Posted by GitBox <gi...@apache.org>.
nastra commented on PR #5529:
URL: https://github.com/apache/iceberg/pull/5529#issuecomment-1219759732

   @rdblue thoughts on enforcing this at compile time?


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