Skip to content

Conversation

@thiagotnunes
Copy link
Contributor

SpannerIO.readChangeStream does not support autoscaling at the moment. In order to avoid customer confusion, we decided to throw an error if any algorithm is specified other than NONE.

SpannerIO.readChangeStream does not support autoscaling at the moment.
In order to avoid customer confusion, we decided to throw an error if
any algorithm is specified other than NONE.
@thiagotnunes
Copy link
Contributor Author

R: @pabloem

@pabloem
Copy link
Member

pabloem commented Mar 29, 2022

Run Java PostCommit

@pabloem
Copy link
Member

pabloem commented Mar 29, 2022

LGTM

@thiagotnunes
Copy link
Contributor Author

Run Java PostCommit

@thiagotnunes
Copy link
Contributor Author

Post Commit failures are unrelated to this PR

@pabloem
Copy link
Member

pabloem commented Mar 29, 2022

Run Java PreCommit

@pabloem
Copy link
Member

pabloem commented Mar 29, 2022

postcommit failures unrelated. LGTM. Merging.

@pabloem pabloem merged commit a090e90 into apache:master Mar 29, 2022
youngoli pushed a commit to youngoli/beam that referenced this pull request Mar 30, 2022
…ng for SpannerIO.readChangeStream

SpannerIO.readChangeStream does not support autoscaling at the moment.
In order to avoid customer confusion, we decided to throw an error if
any algorithm is specified other than NONE.

(cherry picked from commit a090e90)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants