[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-11 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1067649883 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -40,10 +41,16 @@ private[spark] class KVStoreProtobufSerializer

[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-11 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1067650529 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -40,10 +41,16 @@ private[spark] class KVStoreProtobufSerializer

[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-11 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1067650529 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -40,10 +41,16 @@ private[spark] class KVStoreProtobufSerializer

[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-11 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1067647694 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -40,10 +41,16 @@ private[spark] class KVStoreProtobufSerializer

[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-11 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1067649883 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -40,10 +41,16 @@ private[spark] class KVStoreProtobufSerializer

[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-11 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1067649883 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -40,10 +41,16 @@ private[spark] class KVStoreProtobufSerializer

[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-11 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1067650529 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -40,10 +41,16 @@ private[spark] class KVStoreProtobufSerializer

[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-11 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1067649883 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -40,10 +41,16 @@ private[spark] class KVStoreProtobufSerializer

[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-11 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1067647694 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -40,10 +41,16 @@ private[spark] class KVStoreProtobufSerializer

[GitHub] [spark] LuciferYang commented on a diff in pull request #39487: [SPARK-41968][CORE][SQL] Refactor `ProtobufSerDe` to `ProtobufSerDe[T]`

2023-01-10 Thread GitBox
LuciferYang commented on code in PR #39487: URL: https://github.com/apache/spark/pull/39487#discussion_r1066619285 ## core/src/main/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializer.scala: ## @@ -26,7 +27,7 @@ import