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]