[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #39732: [SPARK-42178][UI] Handle remaining null string values in ui protobuf serializer and add tests

2023-01-24 Thread via GitHub
dongjoon-hyun commented on code in PR #39732: URL: https://github.com/apache/spark/pull/39732#discussion_r1086294697 ## core/src/test/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializerSuite.scala: ## @@ -463,7 +492,7 @@ class KVStoreProtobufSerializerSuite extends

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #39732: [SPARK-42178][UI] Handle remaining null string values in ui protobuf serializer and add tests

2023-01-24 Thread via GitHub
dongjoon-hyun commented on code in PR #39732: URL: https://github.com/apache/spark/pull/39732#discussion_r1086292404 ## core/src/test/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializerSuite.scala: ## @@ -463,7 +492,7 @@ class KVStoreProtobufSerializerSuite extends

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #39732: [SPARK-42178][UI] Handle remaining null string values in ui protobuf serializer and add tests

2023-01-24 Thread via GitHub
dongjoon-hyun commented on code in PR #39732: URL: https://github.com/apache/spark/pull/39732#discussion_r1086292024 ## core/src/test/scala/org/apache/spark/status/protobuf/KVStoreProtobufSerializerSuite.scala: ## @@ -31,6 +34,32 @@ import