[GitHub] [iceberg] RussellSpitzer commented on pull request #7607: Spark: Added Scala/Java IcebergMergeInto Api on spark3.4

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

RussellSpitzer commented on PR #7607:
URL: https://github.com/apache/iceberg/pull/7607#issuecomment-1551780147

   First high level comment, I think we really need an underlying Java API as well. I think there should be a first class static class in java that can be used similar to the extension here. Other than that I think the API looks good at first glance, I assume it is matching the Delta api?


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