This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 61a3e0587df6 [SPARK-46312][CORE] Use `lower_camel_case` in
`store_types.proto`
61a3e0587df6 is described below
commit 61a3e0587df6be881cdc115fefb77482fa446b5c
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Thu Dec 7 18:16:21 2023 -0800
[SPARK-46312][CORE] Use `lower_camel_case` in `store_types.proto`
### What changes were proposed in this pull request?
This PR aims to use 'lower_camel_case` in `store_types.proto`.
### Why are the changes needed?
According to our guideline, we had better follow
[FIELD_LOWER_SNAKE_CASE](https://buf.build/docs/lint/rules#field_lower_snake_case)
https://github.com/apache/spark/blob/9585cf6d56e3af37142609668dda1eeda3ec876f/core/src/main/protobuf/org/apache/spark/status/protobuf/store_types.proto#L23
### 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.
Closes #44242 from dongjoon-hyun/SPARK-46312.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.../main/protobuf/org/apache/spark/status/protobuf/store_types.proto | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/core/src/main/protobuf/org/apache/spark/status/protobuf/store_types.proto
b/core/src/main/protobuf/org/apache/spark/status/protobuf/store_types.proto
index 93365add3a64..386c660b16de 100644
--- a/core/src/main/protobuf/org/apache/spark/status/protobuf/store_types.proto
+++ b/core/src/main/protobuf/org/apache/spark/status/protobuf/store_types.proto
@@ -164,7 +164,7 @@ message ExecutorStageSummaryWrapper {
message ExecutorResourceRequest {
optional string resource_name = 1;
int64 amount = 2;
- optional string discoveryScript = 3;
+ optional string discovery_script = 3;
optional string vendor = 4;
}
@@ -277,7 +277,7 @@ message RDDStorageInfoWrapper {
}
message ResourceProfileWrapper {
- ResourceProfileInfo rpInfo = 1;
+ ResourceProfileInfo rp_info = 1;
}
message CachedQuantile {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]