Abyss-lord commented on code in PR #10670: URL: https://github.com/apache/gravitino/pull/10670#discussion_r3090689542
########## clients/client-python/gravitino/client/generic_model_catalog.py: ########## @@ -53,7 +53,7 @@ from gravitino.utils import HTTPClient -class GenericModelCatalog(BaseSchemaCatalog): +class GenericModelCatalog(BaseSchemaCatalog): # pylint: disable=R0901 Review Comment: @jerryshao `R0901`: too-many-ancestors means that the class has too many base classes in its inheritance hierarchy, exceeding pylint’s configured threshold. It indicates that the inheritance chain may be too deep, which can increase complexity and reduce maintainability. For GenericModelCatalog, this warning is not newly introduced by this change. It comes from the existing client abstraction design. This class is built on top of the current catalog/schema/table abstraction hierarchy, and the inheritance chain is a result of reusing existing capabilities, preserving interface consistency, and keeping the implementation aligned with the current architecture. -- 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]
