Skip to content

[SPARK-50098][PYTHON][FOLLOWUP] Update _minimum_googleapis_common_protos_version in setup.py#48643

Closed
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-50098-2
Closed

[SPARK-50098][PYTHON][FOLLOWUP] Update _minimum_googleapis_common_protos_version in setup.py#48643
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-50098-2

Conversation

@dongjoon-hyun
Copy link
Copy Markdown
Member

@dongjoon-hyun dongjoon-hyun commented Oct 24, 2024

What changes were proposed in this pull request?

This is a follow-up of

Why are the changes needed?

To be consistent.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the CIs.

Was this patch authored or co-authored using generative AI tooling?

No.

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

Could you review this follow-up, @viirya ?

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

All tests passed.

@viirya
Copy link
Copy Markdown
Member

viirya commented Oct 24, 2024

Thanks @dongjoon-hyun

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

Thank you, @viirya !

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

This is only a follow-up to make it sure the enforcement logic. The dependency was updated and tested already in the previous PR. Let me merge this~

@dongjoon-hyun dongjoon-hyun deleted the SPARK-50098-2 branch October 24, 2024 23:08
HyukjinKwon added a commit that referenced this pull request Feb 24, 2025
…otos_version in setup.py for pyspark-client

### What changes were proposed in this pull request?

This PR is a followup of #48643 that updates _minimum_googleapis_common_protos_version in setup.py for pyspark-client

### Why are the changes needed?

To match the version with pyspark.

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

No, `pyspark-client` has not been released yet.

### How was this patch tested?

It will be tested in "Debug Build / Spark Connect Python-only (master, Python 3.11) " build.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #50063 from HyukjinKwon/SPARK-50098-followup.

Authored-by: Hyukjin Kwon <gurwls223@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
HyukjinKwon added a commit that referenced this pull request Feb 24, 2025
…otos_version in setup.py for pyspark-client

### What changes were proposed in this pull request?

This PR is a followup of #48643 that updates _minimum_googleapis_common_protos_version in setup.py for pyspark-client

### Why are the changes needed?

To match the version with pyspark.

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

No, `pyspark-client` has not been released yet.

### How was this patch tested?

It will be tested in "Debug Build / Spark Connect Python-only (master, Python 3.11) " build.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #50063 from HyukjinKwon/SPARK-50098-followup.

Authored-by: Hyukjin Kwon <gurwls223@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit a084d64)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 14, 2025
…otos_version in setup.py for pyspark-client

### What changes were proposed in this pull request?

This PR is a followup of apache#48643 that updates _minimum_googleapis_common_protos_version in setup.py for pyspark-client

### Why are the changes needed?

To match the version with pyspark.

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

No, `pyspark-client` has not been released yet.

### How was this patch tested?

It will be tested in "Debug Build / Spark Connect Python-only (master, Python 3.11) " build.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes apache#50063 from HyukjinKwon/SPARK-50098-followup.

Authored-by: Hyukjin Kwon <gurwls223@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit 2151085)
Signed-off-by: Hyukjin Kwon <gurwls223@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.

2 participants