Skip to content

[BEAM-5539] Beam Dependency Update Request: google-cloud-pubsub#7268

Merged
charlesccychen merged 1 commit intoapache:masterfrom
ihji:upgrade_pubsub
Dec 20, 2018
Merged

[BEAM-5539] Beam Dependency Update Request: google-cloud-pubsub#7268
charlesccychen merged 1 commit intoapache:masterfrom
ihji:upgrade_pubsub

Conversation

@ihji
Copy link
Contributor

@ihji ihji commented Dec 12, 2018

updating pubsub python dependency from 0.35.4 to 0.39.


Follow this checklist to help us incorporate your contribution quickly and easily:

  • Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

Post-Commit Tests Status (on master branch)

Lang SDK Apex Dataflow Flink Gearpump Samza Spark
Go Build Status --- --- --- --- --- ---
Java Build Status Build Status Build Status Build Status Build Status Build Status Build Status Build Status
Python Build Status --- Build Status
Build Status
Build Status --- --- ---

@ihji
Copy link
Contributor Author

ihji commented Dec 13, 2018

run portable_python precommit

@ihji
Copy link
Contributor Author

ihji commented Dec 13, 2018

run python postcommit

@ihji
Copy link
Contributor Author

ihji commented Dec 13, 2018

@udim Looks like the pubsub interface hasn't changed since 0.35.4. Is there anything I should check further?

@udim
Copy link
Member

udim commented Dec 13, 2018

No, test coverage in pre and postcommits should be enough.

@udim
Copy link
Member

udim commented Dec 20, 2018

@charlesccychen for merge please

@charlesccychen
Copy link
Contributor

Thanks, this LGTM.

@charlesccychen charlesccychen merged commit 0644c55 into apache:master Dec 20, 2018
@ihji ihji deleted the upgrade_pubsub branch December 27, 2018 21:59
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