Fokko commented on code in PR #5687:
URL: https://github.com/apache/iceberg/pull/5687#discussion_r960633891


##########
python/pyiceberg/cli/console.py:
##########
@@ -17,24 +17,18 @@
 # pylint: disable=broad-except,redefined-builtin,redefined-outer-name
 from functools import wraps
 from typing import (
-    Dict,
     Literal,
     Optional,
     Tuple,
-    Type,
 )
 
 import click
 from click import Context
 
 from pyiceberg.catalog import Catalog, load_catalog
-from pyiceberg.catalog.hive import HiveCatalog
-from pyiceberg.catalog.rest import RestCatalog
 from pyiceberg.cli.output import ConsoleOutput, JsonOutput, Output
 from pyiceberg.exceptions import NoSuchNamespaceError, 
NoSuchPropertyException, NoSuchTableError
 
-SUPPORTED_CATALOGS: Dict[str, Type[Catalog]] = {"thrift": HiveCatalog, "http": 
RestCatalog}

Review Comment:
   This was still lingering here, but isn't needed and causes issues when 
thrift isn't installed.



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to