feat: add version number in .history() and display in reversed chro…

…nological order (#1710)

# Description
Adds the version number to each commit info.

# Related Issue(s)
<!---
For example:

- closes #106 
--->
- Closes #1561
- Closes #1680

---------

Co-authored-by: R. Tyler Croy <[email protected]>

python/deltalake/_internal.pyi

@@ -42,6 +42,7 @@ class RawDeltaTable:
     ) -> str: ...
     def table_uri(self) -> str: ...
     def version(self) -> int: ...
+    def get_latest_version(self) -> int: ...
     def metadata(self) -> RawDeltaTableMetaData: ...
     def protocol_versions(self) -> List[int]: ...
     def load_version(self, version: int) -> None: ...

python/deltalake/table.py

@@ -9,6 +9,7 @@
     TYPE_CHECKING,
     Any,
     Dict,
+    Generator,
     Iterable,
     List,
     NamedTuple,
@@ -428,10 +429,25 @@ def history(self, limit: Optional[int] = None) -> List[Dict[str, Any]]:
         :param limit: the commit info limit to return
         :return: list of the commit infos registered in the transaction log
         """
-        return [
-            json.loads(commit_info_raw)
-            for commit_info_raw in self._table.history(limit)
-        ]
+
+        def _backwards_enumerate(
+            iterable: List[str], start_end: int
+        ) -> Generator[Tuple[int, str], None, None]:
+            n = start_end
+            for elem in iterable:
+                yield n, elem
+                n -= 1
+
+        commits = list(reversed(self._table.history(limit)))
+
+        history = []
+        for version, commit_info_raw in _backwards_enumerate(
+            commits, start_end=self._table.get_latest_version()
+        ):
+            commit = json.loads(commit_info_raw)
+            commit["version"] = version
+            history.append(commit)
+        return history
 
     def vacuum(
         self,

python/src/lib.rs

@@ -171,6 +171,12 @@ impl RawDeltaTable {
             .map_err(PythonError::from)?)
     }
 
+    pub fn get_latest_version(&mut self) -> PyResult<i64> {
+        Ok(rt()?
+            .block_on(self._table.get_latest_version())
+            .map_err(PythonError::from)?)
+    }
+
     pub fn load_with_datetime(&mut self, ds: &str) -> PyResult<()> {
         let datetime =
             DateTime::<Utc>::from(DateTime::<FixedOffset>::parse_from_rfc3339(ds).map_err(

python/tests/test_table_read.py

@@ -299,6 +299,7 @@ def test_history_partitioned_table_metadata():
             "numOutputBytes": "2477",
             "numOutputRows": "7",
         },
+        "version": 0,
     }
 
 

rust/src/table/mod.rs

@@ -415,8 +415,8 @@ impl DeltaTable {
 
         Ok(())
     }
-
-    async fn get_latest_version(&mut self) -> Result<i64, DeltaTableError> {
+    /// returns the latest available version of the table
+    pub async fn get_latest_version(&mut self) -> Result<i64, DeltaTableError> {
         let version_start = match get_last_checkpoint(&self.storage).await {
             Ok(last_check_point) => last_check_point.version,
             Err(ProtocolError::CheckpointNotFound) => {