[GitHub] [iceberg] RussellSpitzer commented on issue #3665: Adding MergeInto into the Spark Scala API

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

RussellSpitzer commented on issue #3665:
URL: https://github.com/apache/iceberg/issues/3665#issuecomment-1456888130

   I do think changing this in Spark is probably the best approach for adding the api to DataframeWriter but I also would be glad to see some code in the Iceberg repo giving us functionality until then. Because we expect this function may be deprecated when the Spark API becomes available, I think it makes sense to keep this as just a public static method rather than an implicit. This should make it a lot clearer when we deprecate and remove it for a Spark native alternative.
   
   Maybe something like
   
   IcebergMergeInto(Table).when(.....)


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