Skip to content

Conversation

@zhengruifeng
Copy link
Contributor

@zhengruifeng zhengruifeng commented Oct 22, 2022

What changes were proposed in this pull request?

pin 'grpcio==1.48.1' 'protobuf==4.21.6'

Why are the changes needed?

to avoid connect-related conflict due to package upgrade

Does this PR introduce any user-facing change?

no

How was this patch tested?

CI

@github-actions github-actions bot added the INFRA label Oct 22, 2022
Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

LGTM if tests pass

@zhengruifeng zhengruifeng changed the title [TEST] pin 'grpcio==1.48.1' 'protobuf==4.21.6' [SPARK-40878][INFRA] pin 'grpcio==1.48.1' 'protobuf==4.21.6' Oct 22, 2022
@zhengruifeng
Copy link
Contributor Author

Merged into master

@zhengruifeng zhengruifeng deleted the infra_pin_connect branch October 22, 2022 04:21
@amaliujia
Copy link
Contributor

amaliujia commented Oct 22, 2022

Post LGTM. Thanks for working on this!

DeZepTup pushed a commit to DeZepTup/spark-custom that referenced this pull request Oct 31, 2022
### What changes were proposed in this pull request?
pin 'grpcio==1.48.1' 'protobuf==4.21.6'

### Why are the changes needed?
to avoid connect-related conflict due to package upgrade

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
CI

Closes apache#38338 from zhengruifeng/infra_pin_connect.

Authored-by: Ruifeng Zheng <ruifengz@apache.org>
Signed-off-by: Ruifeng Zheng <ruifengz@apache.org>
SandishKumarHN pushed a commit to SandishKumarHN/spark that referenced this pull request Dec 12, 2022
### What changes were proposed in this pull request?
pin 'grpcio==1.48.1' 'protobuf==4.21.6'

### Why are the changes needed?
to avoid connect-related conflict due to package upgrade

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
CI

Closes apache#38338 from zhengruifeng/infra_pin_connect.

Authored-by: Ruifeng Zheng <ruifengz@apache.org>
Signed-off-by: Ruifeng Zheng <ruifengz@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants