Skip to content

Revert "update protobuf version to 3.11.4 to match tensorflow-nightly" #1323

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 12, 2021

Conversation

vnghia
Copy link
Contributor

@vnghia vnghia commented Mar 10, 2021

Reverts #1320

The commit tensorflow/tensorflow@e71367f in TensorFlow is reverted by tensorflow/tensorflow@bd3e005. Please wait for tf-nightly 2.5.0.dev20210310 before merging this PR.

Could we freeze the tf-nightly version @yongtang ?

@yongtang
Copy link
Member

@vnvo2409 We can put a specific version in https://github.com/tensorflow/io/blob/master/tensorflow_io/core/python/ops/version_ops.py

e.g.,

package = "tf-nightly==2.5.0.dev20210310"

The build will automatically pin to this version.

We can also wait until TF 2.5 release candidate to pin to rc version instead.

@vnghia vnghia force-pushed the revert-1320-update-protobuf branch from 4643eb6 to edb0fc1 Compare March 10, 2021 15:17
@vnghia
Copy link
Contributor Author

vnghia commented Mar 10, 2021

@yongtang

I believe we need protobuf 3.11.4 for python 3.9, so I will check TensorFlow regularly and ensure that our protobuf match TensorFlow protobuf. I think we could keep updating tf-nightly for now.

In addition, google-cloud-cpp's bigquery will be released in 1 or 2 months I think. I will try to tackle bigtable and pubsub first.

@yongtang
Copy link
Member

@vnvo2409 Thanks! 👍

@vnghia
Copy link
Contributor Author

vnghia commented Mar 10, 2021

@yongtang Tests are falling because of unrelated reasons, we could merge this PR

Copy link
Member

@kvignesh1420 kvignesh1420 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merging the revert as the CI jobs for other PR's are failing.

@kvignesh1420 kvignesh1420 merged commit 7e2eb09 into tensorflow:master Mar 12, 2021
@vnghia vnghia deleted the revert-1320-update-protobuf branch March 12, 2021 10:09
michaelbanfield pushed a commit to michaelbanfield/io that referenced this pull request Mar 30, 2021
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