[GitHub] [iceberg] hailin0 commented on pull request #4649: Flink 1.14: Implement LookupTableSource for IcebergTableSource

[GitBox <gi...@apache.org>]

hailin0 commented on PR #4649:
URL: https://github.com/apache/iceberg/pull/4649#issuecomment-1171830773

   > Hi @hailin0, thanks for your contribution, we all have some concerns about performance and I have a friendly question, have you ever done a test or benchmark for this PR? I am curious about the performance of the point query for Iceberg. AFAIK, Iceberg has some general characteristics with Hive, such that loading all data in memory to accelerate the point query is more reasonable, just like the implementation of Hive Lookup Join, and the community is driving the `[hash lookup join](https://issues.apache.org/jira/browse/FLINK-23687)`, which will bring more benefits, WDYS?
   
   I agree to add the option loading all data in memory


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