Remove MMFiles only feature journalSize

Remove related journal rotation API method and error.

Author: 153957

arango/collection.py

@@ -15,7 +15,6 @@
     CollectionResponsibleShardError,
     CollectionRenameError,
     CollectionRevisionError,
-    CollectionRotateJournalError,
     CollectionStatisticsError,
     CollectionTruncateError,
     CollectionUnloadError,
@@ -307,22 +306,18 @@ def response_handler(resp):
 
         return self._execute(request, response_handler)
 
-    def configure(self, sync=None, journal_size=None):
+    def configure(self, sync=None):
         """Configure collection properties.
 
         :param sync: Block until operations are synchronized to disk.
         :type sync: bool
-        :param journal_size: Journal size in bytes.
-        :type journal_size: int
         :return: New collection properties.
         :rtype: dict
         :raise arango.exceptions.CollectionConfigureError: If operation fails.
         """
         data = {}
         if sync is not None:
             data['waitForSync'] = sync
-        if journal_size is not None:
-            data['journalSize'] = journal_size
 
         request = Request(
             method='put',
@@ -470,25 +465,6 @@ def response_handler(resp):
 
         return self._execute(request, response_handler)
 
-    def rotate(self):
-        """Rotate the collection journal.
-
-        :return: True if collection journal was rotated successfully.
-        :rtype: bool
-        :raise arango.exceptions.CollectionRotateJournalError: If rotate fails.
-        """
-        request = Request(
-            method='put',
-            endpoint='/_api/collection/{}/rotate'.format(self.name),
-        )
-
-        def response_handler(resp):
-            if not resp.is_success:
-                raise CollectionRotateJournalError(resp, request)
-            return True  # pragma: no cover
-
-        return self._execute(request, response_handler)
-
     def truncate(self):
         """Delete all documents in the collection.
 

arango/database.py

@@ -949,7 +949,6 @@ def create_collection(self,
                           name,
                           sync=False,
                           system=False,
-                          journal_size=None,
                           edge=False,
                           user_keys=True,
                           key_increment=None,
@@ -974,8 +973,6 @@ def create_collection(self,
         :param system: If set to True, a system collection is created. The
             collection name must have leading underscore "_" character.
         :type system: bool
-        :param journal_size: Max size of the journal in bytes.
-        :type journal_size: int
         :param edge: If set to True, an edge collection is created.
         :type edge: bool
         :param key_generator: Used for generating document keys. Allowed values
@@ -1055,8 +1052,6 @@ def create_collection(self,
             'keyOptions': key_options,
             'type': 3 if edge else 2
         }
-        if journal_size is not None:
-            data['journalSize'] = journal_size
         if shard_count is not None:
             data['numberOfShards'] = shard_count
         if shard_fields is not None:

arango/exceptions.py

@@ -245,10 +245,6 @@ class CollectionUnloadError(ArangoServerError):
     """Failed to unload collection."""
 
 
-class CollectionRotateJournalError(ArangoServerError):
-    """Failed to rotate collection journal."""
-
-
 class CollectionRecalculateCountError(ArangoServerError):
     """Failed to recalculate document count."""
 

docs/collection.rst

@@ -46,7 +46,7 @@ Here is an example showing how you can manage standard collections:
     students.load()
     students.unload()
     students.truncate()
-    students.configure(journal_size=3000000)
+    students.configure()
 
     # Delete the collection.
     db.delete_collection('students')

tests/test_collection.py

@@ -45,15 +45,14 @@ def test_collection_misc_methods(col, bad_col, cluster):
     prev_sync = properties['sync']
     properties = col.configure(
         sync=not prev_sync,
-        journal_size=10000000
     )
     assert properties['name'] == col.name
     assert properties['system'] is False
     assert properties['sync'] is not prev_sync
 
     # Test configure properties with bad collection
     with assert_raises(CollectionConfigureError) as err:
-        bad_col.configure(sync=True, journal_size=10000000)
+        bad_col.configure(sync=True)
     assert err.value.error_code in {11, 1228}
 
     # Test get statistics
@@ -151,7 +150,6 @@ def test_collection_management(db, bad_db, cluster):
     col = db.create_collection(
         name=col_name,
         sync=True,
-        journal_size=7774208,
         system=False,
         key_generator='traditional',
         user_keys=False,