Skip to content

Conversation

@keelerm84
Copy link
Member

@keelerm84 keelerm84 commented Jan 16, 2026

Note

Removes the legacy "disable streaming" warning and aligns tests with the new logging behavior.

  • Delete warning log from PollingDataSourceBuilderV2.Build that discouraged disabling streaming
  • Update FDv2 end-to-end tests to stop asserting pollingModeWarningMessage warnings; keep other error/warn assertions intact
  • Minor test cleanup (whitespace/handler setup)

Written by Cursor Bugbot for commit 34fa4d0. This will update automatically on new commits. Configure here.

@keelerm84 keelerm84 requested a review from a team as a code owner January 16, 2026 19:27
@keelerm84 keelerm84 changed the title chore: Remove disable-streaming warning fix: Remove disable streaming warning Jan 16, 2026
@keelerm84 keelerm84 merged commit 6fc5062 into v7 Jan 16, 2026
28 checks passed
@keelerm84 keelerm84 deleted the mk/remove-fdv2-polling-warning branch January 16, 2026 20:29
keelerm84 pushed a commit that referenced this pull request Jan 16, 2026
🤖 I have created a release *beep* *boop*
---


##
[7.14.5](v7.14.4...v7.14.5)
(2026-01-16)


### Bug Fixes

* Remove disable streaming warning
([#340](#340))
([6fc5062](6fc5062))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Releases v7.14.5 with a bug fix and version bumps.
> 
> - Update `CHANGELOG.md` for 7.14.5, noting bug fix: remove disable
streaming warning
> - Bump SDK version from `7.14.4` to `7.14.5` in
`.release-please-manifest.json` and `internal/version.go`
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
0c8ca32. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants