Skip to content

Commit

Permalink
Adding rudimentary support for paginating the results returned by the…
Browse files Browse the repository at this point in the history
… Synapse monitoring client

Fixes Azure#19855

Using kwargs for now to avoid changing the method APIs.

Issues not yet addressed:

- [ ] more complex filtering support
- [ ] pagination sizes (100 was inferred from traceing the web page requests for testing)
  • Loading branch information
bpkroth committed Jul 16, 2021
1 parent 22cc32f commit 4b06279
Showing 1 changed file with 6 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,12 @@ def get_spark_job_list(
query_parameters = {} # type: Dict[str, Any]
query_parameters['api-version'] = self._serialize.query("api_version", api_version, 'str')

# Add rudimentary support for paginating the results returned:
# See Also: <https://github.com/Azure/azure-sdk-for-python/issues/19855>
skip = kwargs.pop('skip', None)
if skip != None:
query_parameters['skip'] = self._serialize.query("skip", skip, 'int')

# Construct headers
header_parameters = {} # type: Dict[str, Any]
if x_ms_client_request_id is not None:
Expand Down

0 comments on commit 4b06279

Please sign in to comment.