yuqi1129 commented on code in PR #9160: URL: https://github.com/apache/gravitino/pull/9160#discussion_r2559255122
########## catalogs/catalog-lakehouse-generic/src/main/java/org/apache/gravitino/catalog/lakehouse/generic/GenericCatalog.java: ########## Review Comment: > I'm concerned about the catalog provider name. I agree on this point, the name of the provider seems to be more important compared to the class name, and we'd better make these two consistent. If the provider name is `lakehouse_catalog`, then I would like to adopt your suggestion and use `LakehouseCatalog` as the name. There is no need to struggling with the name of the class name as it can be changed later without any compatiable issues. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
