From b450fe9d42a5f404d8daadb62d291eed1b66659c Mon Sep 17 00:00:00 2001
From: Yufei Gu <yufei.apache.org>
Date: Wed, 22 Oct 2025 13:32:00 -0700
Subject: [PATCH] Remove the trailing comma

---
 client/python/cli/command/__init__.py  |  2 +-
 client/python/test/test_cli_parsing.py | 13 ++++++++++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/client/python/cli/command/__init__.py b/client/python/cli/command/__init__.py
index 4463649c29..8a7d3d84c7 100644
--- a/client/python/cli/command/__init__.py
+++ b/client/python/cli/command/__init__.py
@@ -40,7 +40,7 @@ def options_get(key, f=lambda x: x):
         set_properties = Parser.parse_properties(options_get(Arguments.SET_PROPERTY))
         remove_properties = options_get(Arguments.REMOVE_PROPERTY)
         catalog_client_scopes = options_get(Arguments.CATALOG_CLIENT_SCOPE)
-        parameters = Parser.parse_properties(options_get(Arguments.PARAMETERS)),
+        parameters = Parser.parse_properties(options_get(Arguments.PARAMETERS))
 
         command = None
         if options.command == Commands.CATALOGS:
diff --git a/client/python/test/test_cli_parsing.py b/client/python/test/test_cli_parsing.py
index 1cca75e2fa..af5f8e28b3 100644
--- a/client/python/test/test_cli_parsing.py
+++ b/client/python/test/test_cli_parsing.py
@@ -618,6 +618,17 @@ def get(obj, arg_string):
                 (1, 'client_secret'): 'e469c048cf866dfae469c048cf866df1',
             })
 
+    def test_policies_attach_parameters_parsed_to_dict(self):
+        options = Parser.parse([
+            'policies', 'attach', 'policy-name',
+            '--catalog', 'cat',
+            '--attachment-type', 'catalog',
+            '--parameters', 'key=value',
+        ])
+        command = Command.from_options(options)
+        self.assertIsInstance(command.parameters, dict)
+        self.assertEqual({'key': 'value'}, command.parameters)
+
 
 if __name__ == '__main__':
-    unittest.main()
\ No newline at end of file
+    unittest.main()