Skip to content

Commit

Permalink
[SPARK-46293][CONNECT][PYTHON] Use protobuf transitive dependency
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

This PR proposes to remove `protobuf` from required package.

### Why are the changes needed?

`protobuf` is automatically installed when installing `grpcio` and `grpcio-status`, so we don't need to specify the specific version explicitly.

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

No API changes.

### How was this patch tested?

The existing CI should pass

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

No.

Closes apache#44221 from itholic/protobuf_docs.

Authored-by: Haejoon Lee <haejoon.lee@databricks.com>
Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
  • Loading branch information
itholic authored and dongjoon-hyun committed Dec 7, 2023
1 parent 8132e17 commit e359318
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion dev/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,6 @@ py
# Spark Connect (required)
grpcio>=1.59.3
grpcio-status>=1.59.3
protobuf==4.25.1
googleapis-common-protos>=1.56.4

# Spark Connect python proto generation plugin (optional)
Expand Down

0 comments on commit e359318

Please sign in to comment.