You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@polaris.apache.org by yz...@apache.org on 2026/07/26 01:54:47 UTC

(polaris) branch main updated: CLI: Fix dead code (#5157)

This is an automated email from the ASF dual-hosted git repository.

MonkeyCanCode pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new a6681ba115 CLI: Fix dead code (#5157)
a6681ba115 is described below

commit a6681ba115c6fcc77ced89fe9024f2cb71fa75d9
Author: Yong Zheng <yo...@gmail.com>
AuthorDate: Sat Jul 25 20:54:43 2026 -0500

    CLI: Fix dead code (#5157)
    
    * Remove dead code in find command
    
    * Fix dead code in parser tests
---
 client/python/apache_polaris/cli/command/find.py | 2 --
 client/python/tests/test_parser_basic.py         | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/client/python/apache_polaris/cli/command/find.py b/client/python/apache_polaris/cli/command/find.py
index 5799a2f626..0606c3c2b4 100644
--- a/client/python/apache_polaris/cli/command/find.py
+++ b/client/python/apache_polaris/cli/command/find.py
@@ -96,8 +96,6 @@ class FindCommand(Command):
                     )
             elif self.catalog_name:
                 print("No catalogs found to search.")
-        else:
-            catalogs_to_search = []
 
         # Summary
         total_results = sum(self._type_counts.values())
diff --git a/client/python/tests/test_parser_basic.py b/client/python/tests/test_parser_basic.py
index bab24912e1..6764239222 100644
--- a/client/python/tests/test_parser_basic.py
+++ b/client/python/tests/test_parser_basic.py
@@ -73,7 +73,7 @@ class TestParserBasic(CLITestBase):
                     "gone",
                 ]
             )  # remote-url deprecated
-            self.assertEqual(cm.exception.code, INVALID_ARGS)
+        self.assertEqual(cm.exception.code, INVALID_ARGS)
 
         with self.assertRaises(SystemExit) as cm:
             Parser.parse(["principals", "create", "name", "--type", "bad"])