feat: enhance SQL pagination handling with warnings for infinite loops and update tests

Author: Samson Gebre

CHANGELOG.md

@@ -7,8 +7,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Added
+- Batch API: `client.batch` namespace for deferred-execution batch operations that pack multiple Dataverse Web API calls into a single `POST $batch` HTTP request (#129)
+- Batch DataFrame integration: `client.batch.dataframe` namespace with pandas DataFrame wrappers for batch operations (#129)
+- `client.records.upsert()` and `client.batch.records.upsert()` backed by the `UpsertMultiple` bound action with alternate-key support (#129)
+- QueryBuilder: `client.query.builder("table")` with a fluent API, 20+ chainable methods (`select`, `filter_eq`, `filter_contains`, `order_by`, `expand`, etc.), and composable filter expressions using Python operators (`&`, `|`, `~`) (#118)
+- Memo/multiline column type support: `"memo"` (or `"multiline"`) can now be passed as a column type in `client.tables.create()` and `client.tables.add_columns()` (#155)
+
+### Changed
+- Picklist label-to-integer resolution now uses a single bulk `PicklistAttributeMetadata` API call for the entire table instead of per-attribute requests, with a 1-hour TTL cache (#154)
+
 ### Fixed
 - `client.query.sql()` silently truncated results at 5,000 rows. The method now follows `@odata.nextLink` pagination and returns all matching rows (#157).
+- Alternate key fields were incorrectly merged into the `UpsertMultiple` request body, causing `400 Bad Request` on the create path (#129)
+- Docstring type annotations corrected for Microsoft Learn API reference compatibility (#153)
 
 ## [0.1.0b7] - 2026-03-17
 
@@ -96,6 +108,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Comprehensive error handling with specific exception types (`DataverseError`, `AuthenticationError`, etc.) (#22, #24)
 - HTTP retry logic with exponential backoff for resilient operations (#72)
 
+[Unreleased]: https://github.com/microsoft/PowerPlatform-DataverseClient-Python/compare/v0.1.0b7...HEAD
 [0.1.0b7]: https://github.com/microsoft/PowerPlatform-DataverseClient-Python/compare/v0.1.0b6...v0.1.0b7
 [0.1.0b6]: https://github.com/microsoft/PowerPlatform-DataverseClient-Python/compare/v0.1.0b5...v0.1.0b6
 [0.1.0b5]: https://github.com/microsoft/PowerPlatform-DataverseClient-Python/compare/v0.1.0b4...v0.1.0b5

src/PowerPlatform/Dataverse/data/_odata.py

@@ -825,6 +825,15 @@ def _query_sql(self, sql: str) -> list[dict[str, Any]]:
         while next_link:
             # Guard 1: exact URL cycle (same next_link returned twice)
             if next_link in visited:
+                warnings.warn(
+                    f"SQL pagination stopped after {len(results)} rows — "
+                    "the Dataverse server returned the same nextLink URL twice, "
+                    "indicating an infinite pagination cycle. "
+                    "Returning the rows collected so far. "
+                    "To avoid pagination entirely, add a TOP clause to your query.",
+                    RuntimeWarning,
+                    stacklevel=4,
+                )
                 break
             visited.add(next_link)
             # Guard 2: server-side bug where pagingcookie does not advance between

tests/unit/data/test_sql_parse.py

@@ -225,8 +225,9 @@ def test_query_sql_mid_pagination_error_warns_and_returns_partial():
     assert result == [{"id": 1}]
 
 
-def test_query_sql_repeated_next_link_stops_pagination():
-    """If the server keeps returning the same @odata.nextLink the loop must not run forever."""
+def test_query_sql_repeated_next_link_warns_and_stops():
+    """If the server keeps returning the same @odata.nextLink a RuntimeWarning is emitted and
+    the loop stops without running forever."""
     client = _query_sql_client()
     # Both pages return the same next_link — simulates a server that re-executes the SQL
     repeating_body = {"value": [{"id": 1}], "@odata.nextLink": "https://org.example/page2"}
@@ -241,7 +242,8 @@ def test_query_sql_repeated_next_link_stops_pagination():
         patch.object(client, "_build_sql", return_value=MagicMock()),
         patch.object(client, "_request", return_value=resp2) as mock_req,
     ):
-        result = client._query_sql("SELECT id FROM account")
+        with pytest.warns(RuntimeWarning, match="pagination stopped"):
+            result = client._query_sql("SELECT id FROM account")
 
     # fetched page2 once, then detected the cycle and stopped
     mock_req.assert_called_once_with("get", "https://org.example/page2")
@@ -343,11 +345,11 @@ def _make_next_link(pagingcookie_inner: str, pagenumber: int = 2) -> str:
     from urllib.parse import quote as _url_quote
 
     skiptoken_xml = (
-        f'<cookie pagenumber="{pagenumber}" '
-        f'pagingcookie="{pagingcookie_inner}" '
-        f'istracking="False" />'
+        f'<cookie pagenumber="{pagenumber}" ' f'pagingcookie="{pagingcookie_inner}" ' f'istracking="False" />'
+    )
+    return (
+        f"https://org.example/api/data/v9.2?$sql=SELECT%20name%20FROM%20account&$skiptoken={_url_quote(skiptoken_xml)}"
     )
-    return f"https://org.example/api/data/v9.2?$sql=SELECT%20name%20FROM%20account&$skiptoken={_url_quote(skiptoken_xml)}"
 
 
 def test_extract_pagingcookie_returns_cookie_value():