[GitHub] [iceberg] omarsmak edited a comment on issue #3289: Include Nessie as a catalog impl as a known type

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

omarsmak edited a comment on issue #3289:
URL: https://github.com/apache/iceberg/issues/3289#issuecomment-942106736


   Totally valid point and I do agree with you about this. However, `hive` and `hadoop` catalogs are deeply integrated in iceberg core and thus, introducing the type was necessary (also for some core functionalities to distinguish between both). 
   If we opt to introduce types for custom catalogs, IMHO this shouldn't be only for `nessie` but should be for every custom catalog lives in iceberg codebase. 
   @rdblue any thoughts on this?


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