#3509 refactor: Deprecate `RESTStream.prepare_request` in favor of `RESTStream.get_http_request`
Pull request opened by
edgarrmondragon
SSIA
Summary by Sourcery
Deprecate RESTStream.prepare_request in favor of RESTStream.get_http_request and update pagination handling to operate on paginator instances instead of raw page tokens.
New Features:
• Allow HTTPRequestContext to carry a paginator instance, enabling richer pagination-aware request construction.
• Expose the page_size attribute on BaseOffsetPaginator for use in request parameter building.
Enhancements:
• Adjust REST stream request construction to use paginator.current_value via the new HTTPRequestContext paginator field.
• Introduce an internal helper to build PreparedRequest objects from HTTPRequestContext while keeping backward compatibility with overridden prepare_request implementations.
• Tighten typing on get_new_paginator and related HTTPRequestContext generics to bind them to concrete paginator types across the SDK and example taps.
• Update example taps and tests to construct HTTP requests using paginator instances rather than raw next_page tokens.
Tests:
• Adapt REST pagination tests to the new HTTPRequestContext paginator API and ensure backward compatibility paths are covered.
meltano/sdk