Skip to content

Commit

Permalink
Address comments
Browse files Browse the repository at this point in the history
  • Loading branch information
sfc-gh-melnacouzi committed Dec 3, 2024
1 parent fce7d76 commit b9c8c05
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions src/snowflake/cli/_plugins/connection/util.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,8 @@ def get_ui_parameters(conn: SnowflakeConnection) -> Dict[UIParameter, Any]:

parameters_to_fetch = [param.value for param in UIParameter]

# Parsing of the Json and the filtering is happening here in Snowflake CLI
# in order to avoid requiring a warehouse in Snowflake
query = "call system$bootstrap_data_request('CLIENT_PARAMS_INFO')"
*_, cursor = conn.execute_string(query)

Expand Down
4 changes: 2 additions & 2 deletions src/snowflake/cli/api/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -286,10 +286,10 @@ def get_config_value(*path, key: str, default: Optional[Any] = Empty) -> Any:
raise


def get_config_bool_value(*path, key: str, default: Any = Empty) -> bool | None:
def get_config_bool_value(*path, key: str, default: Optional[bool]) -> Optional[bool]:
value = get_config_value(*path, key=key, default=None)

if value is None and default is not Empty:
if value is None:
return default

try:
Expand Down

0 comments on commit b9c8c05

Please sign in to comment.