diff --git a/client/python/cli/command/catalogs.py b/client/python/cli/command/catalogs.py index 9f7cd2015b..718e6d11aa 100644 --- a/client/python/cli/command/catalogs.py +++ b/client/python/cli/command/catalogs.py @@ -16,15 +16,15 @@ # specific language governing permissions and limitations # under the License. # -from dataclasses import dataclass, field -from typing import Dict, Optional, List +from dataclasses import dataclass +from typing import Dict, List from pydantic import StrictStr from cli.command import Command from cli.constants import StorageType, CatalogType, Subcommands, Arguments from cli.options.option_tree import Argument -from polaris.management import PolarisDefaultApi, Catalog, CreateCatalogRequest, UpdateCatalogRequest, \ +from polaris.management import PolarisDefaultApi, CreateCatalogRequest, UpdateCatalogRequest, \ StorageConfigInfo, ExternalCatalog, AwsStorageConfigInfo, AzureStorageConfigInfo, GcpStorageConfigInfo, \ PolarisCatalog, CatalogProperties diff --git a/client/python/cli/command/namespaces.py b/client/python/cli/command/namespaces.py index 9764016995..65687226a0 100644 --- a/client/python/cli/command/namespaces.py +++ b/client/python/cli/command/namespaces.py @@ -27,7 +27,6 @@ from cli.constants import Subcommands, Arguments, UNIT_SEPARATOR from cli.options.option_tree import Argument from polaris.catalog import IcebergCatalogAPI, CreateNamespaceRequest, ApiClient, Configuration -from polaris.catalog.exceptions import NotFoundException from polaris.management import PolarisDefaultApi diff --git a/client/python/cli/command/principal_roles.py b/client/python/cli/command/principal_roles.py index c64e363f02..911f741f5c 100644 --- a/client/python/cli/command/principal_roles.py +++ b/client/python/cli/command/principal_roles.py @@ -25,7 +25,7 @@ from cli.constants import Subcommands, Arguments from cli.options.option_tree import Argument from polaris.management import PolarisDefaultApi, CreatePrincipalRoleRequest, PrincipalRole, UpdatePrincipalRoleRequest, \ - GrantCatalogRoleRequest, CatalogRole, GrantPrincipalRoleRequest + GrantPrincipalRoleRequest @dataclass diff --git a/client/python/cli/command/profiles.py b/client/python/cli/command/profiles.py index ed0a102818..8679907b0d 100644 --- a/client/python/cli/command/profiles.py +++ b/client/python/cli/command/profiles.py @@ -22,7 +22,6 @@ from dataclasses import dataclass from typing import Dict, Optional, List -from pydantic import StrictStr from cli.command import Command from cli.constants import Subcommands, DEFAULT_HOSTNAME, DEFAULT_PORT, CONFIG_DIR, CONFIG_FILE diff --git a/client/python/cli/options/parser.py b/client/python/cli/options/parser.py index 6413096803..8613474fe7 100644 --- a/client/python/cli/options/parser.py +++ b/client/python/cli/options/parser.py @@ -68,7 +68,7 @@ def add_arguments(parser, args: List[Argument]): if arg.default: kwargs['default'] = arg.default - if arg.type == bool: + if arg.type is bool: del kwargs['type'] parser.add_argument(arg.get_flag_name(), **kwargs, action='store_true') elif arg.allow_repeats: @@ -128,7 +128,7 @@ def parse_args(self, args=None, namespace=None): tree_str = self._get_tree_str(args[:help_index]) if tree_str: print(f'input: polaris {" ".join(args)}') - print(f'options:') + print('options:') print(tree_str) print('\n') self.print_usage() @@ -187,7 +187,7 @@ def _get_command_path(self, args: List[str], options: List[Option]) -> List[str] parser = parser._subparsers._group_actions[0].choices.get(arg) if not parser: break - except Exception as e: + except Exception: break options = list(filter(lambda o: o.name == arg, options))[0].children if options is None: diff --git a/client/python/cli/polaris_cli.py b/client/python/cli/polaris_cli.py index b4d1fd8e41..d607912326 100644 --- a/client/python/cli/polaris_cli.py +++ b/client/python/cli/polaris_cli.py @@ -133,7 +133,7 @@ def _get_client_builder(options): if options.host is not None or options.port is not None: raise Exception(f'Please provide either {Argument.to_flag_name(Arguments.BASE_URL)} or' f' {Argument.to_flag_name(Arguments.HOST)} &' - f' {Argument.to_flag_name(Arguments.PORT)}, but not both'); + f' {Argument.to_flag_name(Arguments.PORT)}, but not both') polaris_management_url = f'{options.base_url}/api/management/v1' polaris_catalog_url = f'{options.base_url}/api/catalog/v1'