Skip to content

Commit afbeafd

Browse files
author
Saurabh Badenkal
committed
Fix mypy error in delete(), restore __init__ exports, add large-batch docstring tips
1 parent d028005 commit afbeafd

2 files changed

Lines changed: 14 additions & 6 deletions

File tree

src/PowerPlatform/Dataverse/operations/__init__.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,4 @@
88
SDK operations into logical groups: records, query, and tables.
99
"""
1010

11-
from .dataframe import DataFrameOperations
12-
13-
__all__ = ["DataFrameOperations"]
11+
__all__: list[str] = []

src/PowerPlatform/Dataverse/operations/dataframe.py

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -154,6 +154,11 @@ def create(
154154
:raises ValueError: If ``records`` is empty or the number of returned
155155
IDs does not match the number of input rows.
156156
157+
.. tip::
158+
All rows are sent in a single ``CreateMultiple`` request. For very
159+
large DataFrames, consider splitting into smaller batches to avoid
160+
request timeouts.
161+
157162
Example:
158163
Create records from a DataFrame::
159164
@@ -208,6 +213,11 @@ def update(
208213
:raises TypeError: If ``changes`` is not a pandas DataFrame.
209214
:raises ValueError: If ``id_column`` is not found in the DataFrame.
210215
216+
.. tip::
217+
All rows are sent in a single ``UpdateMultiple`` request (or a
218+
single PATCH for one row). For very large DataFrames, consider
219+
splitting into smaller batches to avoid request timeouts.
220+
211221
Example:
212222
Update records with different values per row::
213223
@@ -300,6 +310,6 @@ def delete(
300310
)
301311

302312
if len(id_list) == 1:
303-
return self._client.records.delete(table, id_list[0])
304-
else:
305-
return self._client.records.delete(table, id_list, use_bulk_delete=use_bulk_delete)
313+
self._client.records.delete(table, id_list[0])
314+
return None
315+
return self._client.records.delete(table, id_list, use_bulk_delete=use_bulk_delete)

0 commit comments

Comments
 (0)