Skip to content

Conversation

@keelerm84
Copy link
Member

Convert feature requester and polling data source to use the generic HttpTransport trait instead of hyper Client directly.

  • Replace HyperFeatureRequester with HttpFeatureRequester
  • Replace HyperFeatureRequesterBuilder with HttpFeatureRequesterBuilder
  • Update PollingDataSourceBuilder to accept transport
  • Stream and collect response body for JSON parsing
  • Remove hyper-specific imports and trait bounds
  • Update test helpers to use HyperTransport

BREAKING CHANGE: PollingDataSourceBuilder generic parameter
changed from connector (C) to transport (T: HttpTransport).

@keelerm84 keelerm84 requested a review from a team as a code owner January 13, 2026 15:47
@keelerm84 keelerm84 force-pushed the mk/sdk-1744/3-migrate-event-and-polling branch from 5ddbbd3 to a08b9e0 Compare January 20, 2026 18:31
Base automatically changed from mk/sdk-1744/3-migrate-event-and-polling to feat/3.x January 20, 2026 18:38
Convert feature requester and polling data source to use the
generic HttpTransport trait instead of hyper Client directly.

- Replace HyperFeatureRequester<C> with HttpFeatureRequester<T>
- Replace HyperFeatureRequesterBuilder with HttpFeatureRequesterBuilder
- Update PollingDataSourceBuilder to accept transport
- Stream and collect response body for JSON parsing
- Remove hyper-specific imports and trait bounds
- Update test helpers to use HyperTransport

BREAKING CHANGE: PollingDataSourceBuilder generic parameter
changed from connector (C) to transport (T: HttpTransport).

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@keelerm84 keelerm84 force-pushed the mk/sdk-1745/4-migrate-polling branch from 2d20295 to 734a306 Compare January 20, 2026 18:40
@keelerm84 keelerm84 merged commit 92630fb into feat/3.x Jan 20, 2026
5 checks passed
@keelerm84 keelerm84 deleted the mk/sdk-1745/4-migrate-polling branch January 20, 2026 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants