Re: [PR] Fix CLI command order and catalog not found error [iceberg-python]

2025-03-25 Thread via GitHub
jason810496 commented on code in PR #1828: URL: https://github.com/apache/iceberg-python/pull/1828#discussion_r2008723316 ## pyiceberg/cli/console.py: ## @@ -86,31 +91,48 @@ def run( ctx.obj["output"] = JsonOutput(verbose=verbose) try: -ctx.obj["catalog"]

Re: [PR] Fix CLI command order and catalog not found error [iceberg-python]

2025-03-22 Thread via GitHub
jason810496 commented on code in PR #1828: URL: https://github.com/apache/iceberg-python/pull/1828#discussion_r2008723316 ## pyiceberg/cli/console.py: ## @@ -86,31 +91,48 @@ def run( ctx.obj["output"] = JsonOutput(verbose=verbose) try: -ctx.obj["catalog"]

Re: [PR] Fix CLI command order and catalog not found error [iceberg-python]

2025-03-22 Thread via GitHub
jason810496 commented on code in PR #1828: URL: https://github.com/apache/iceberg-python/pull/1828#discussion_r2008723316 ## pyiceberg/cli/console.py: ## @@ -86,31 +91,48 @@ def run( ctx.obj["output"] = JsonOutput(verbose=verbose) try: -ctx.obj["catalog"]

Re: [PR] Fix CLI command order and catalog not found error [iceberg-python]

2025-03-22 Thread via GitHub
jason810496 commented on code in PR #1828: URL: https://github.com/apache/iceberg-python/pull/1828#discussion_r2008722936 ## pyiceberg/cli/console.py: ## @@ -86,31 +91,48 @@ def run( ctx.obj["output"] = JsonOutput(verbose=verbose) try: -ctx.obj["catalog"]

Re: [PR] Fix CLI command order and catalog not found error [iceberg-python]

2025-03-22 Thread via GitHub
jason810496 commented on code in PR #1828: URL: https://github.com/apache/iceberg-python/pull/1828#discussion_r200870 ## pyiceberg/cli/console.py: ## @@ -86,31 +91,48 @@ def run( ctx.obj["output"] = JsonOutput(verbose=verbose) try: -ctx.obj["catalog"]

[PR] Fix CLI command order and catalog not found error [iceberg-python]

2025-03-22 Thread via GitHub
iting0321 opened a new pull request, #1828: URL: https://github.com/apache/iceberg-python/pull/1828 Closes #1784 # Rationale for this change The command order for `pyiceberg list --catalog hive` would raise error; it should be `pyiceberg --catalog hive list`. Now, `pyiceberg l