[python] rename drop_database/drop_table ignore_if_exists flag to ignore_if_not_exists (#6846)

Author: plusplusjiajia

paimon-python/pypaimon/catalog/rest/rest_catalog.py

@@ -120,11 +120,11 @@ def get_database(self, name: str) -> Database:
         if response is not None:
             return Database(name, options)
 
-    def drop_database(self, name: str, ignore_if_exists: bool = False):
+    def drop_database(self, name: str, ignore_if_not_exists: bool = False):
         try:
             self.rest_api.drop_database(name)
         except NoSuchResourceException as e:
-            if not ignore_if_exists:
+            if not ignore_if_not_exists:
                 # Convert REST API exception to catalog exception
                 raise DatabaseNotExistException(name) from e
 
@@ -168,13 +168,13 @@ def create_table(self, identifier: Union[str, Identifier], schema: Schema, ignor
             if not ignore_if_exists:
                 raise TableAlreadyExistException(identifier) from e
 
-    def drop_table(self, identifier: Union[str, Identifier], ignore_if_exists: bool = False):
+    def drop_table(self, identifier: Union[str, Identifier], ignore_if_not_exists: bool = False):
         if not isinstance(identifier, Identifier):
             identifier = Identifier.from_string(identifier)
         try:
             self.rest_api.drop_table(identifier)
         except NoSuchResourceException as e:
-            if not ignore_if_exists:
+            if not ignore_if_not_exists:
                 raise TableNotExistException(identifier) from e
 
     def load_table_metadata(self, identifier: Identifier) -> TableMetadata:

paimon-python/pypaimon/tests/py36/ao_simple_test.py

@@ -380,7 +380,7 @@ def test_create_drop_database_table(self):
         try:
             self.rest_catalog.drop_table("db1.tbl1", True)
         except TableNotExistException:
-            self.fail("drop_table with ignore_if_exists=True should not raise TableNotExistException")
+            self.fail("drop_table with ignore_if_not_exists=True should not raise TableNotExistException")
 
         # test drop database
         self.rest_catalog.drop_database("db1", False)
@@ -391,7 +391,7 @@ def test_create_drop_database_table(self):
         try:
             self.rest_catalog.drop_database("db1", True)
         except DatabaseNotExistException:
-            self.fail("drop_database with ignore_if_exists=True should not raise DatabaseNotExistException")
+            self.fail("drop_database with ignore_if_not_exists=True should not raise DatabaseNotExistException")
 
     def test_initialize_oss_fs_pyarrow_lt_7(self):
         props = {

paimon-python/pypaimon/tests/rest/rest_simple_test.py

@@ -698,7 +698,7 @@ def test_create_drop_database_table(self):
         try:
             self.rest_catalog.drop_table("db1.tbl1", True)
         except TableNotExistException:
-            self.fail("drop_table with ignore_if_exists=True should not raise TableNotExistException")
+            self.fail("drop_table with ignore_if_not_exists=True should not raise TableNotExistException")
 
         # test drop database
         self.rest_catalog.drop_database("db1", False)
@@ -709,4 +709,4 @@ def test_create_drop_database_table(self):
         try:
             self.rest_catalog.drop_database("db1", True)
         except DatabaseNotExistException:
-            self.fail("drop_database with ignore_if_exists=True should not raise DatabaseNotExistException")
+            self.fail("drop_database with ignore_if_not_exists=True should not raise DatabaseNotExistException")