You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "dramaticlly (via GitHub)" <gi...@apache.org> on 2026/08/12 23:10:22 UTC

[PR] Flink: Remove deprecated RewriteDataFiles.Builder.filter(Expression) [iceberg]

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

   Deprecated for removal in 1.12.0 in favor of
   filter(SerializableSupplier<Expression>). Updated the one test caller in each version to pass a supplier.
   
   Applies to Flink 1.20, 2.0 and 2.1.
   
   FYI @Guosmilesmile if you want to check 


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


Re: [PR] Flink: Remove deprecated RewriteDataFiles.Builder.filter(Expression) [iceberg]

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

   Merged to main.
   Thanks @dramaticlly for taking care of the deprecation!


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


Re: [PR] Flink: Remove deprecated RewriteDataFiles.Builder.filter(Expression) [iceberg]

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

   +1, thank you @dramaticlly!


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


Re: [PR] Flink: Remove deprecated RewriteDataFiles.Builder.filter(Expression) [iceberg]

Posted by "pvary (via GitHub)" <gi...@apache.org>.
pvary merged PR #17624:
URL: https://github.com/apache/iceberg/pull/17624


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