[GitHub] [iceberg] omarsmak commented on pull request #3162: Core: Throw an exception if both catalog type and catalog-impl are set

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

omarsmak commented on pull request #3162:
URL: https://github.com/apache/iceberg/pull/3162#issuecomment-940126141


   @rdblue I have moved the logic to `CatalogUtil` instead of `Catalog`. This will work for both `Spark` and `Hive`. For Flink logic will be very similar to `CatalogUtil` but in `FlinkCatalogFactory`. If you think what I did here it makes sense for Hive and Spark, I will add Flink as well with the same logic .


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