[GitHub] [spark] justaparth commented on a diff in pull request #41498: [SPARK-44001][Protobuf] spark protobuf: handle well known wrapper types

2023-06-08 Thread via GitHub
justaparth commented on code in PR #41498: URL: https://github.com/apache/spark/pull/41498#discussion_r1223202896 ## connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/ProtobufDeserializer.scala: ## @@ -247,12 +247,86 @@ private[sql] class ProtobufDeserializer(

[GitHub] [spark] justaparth commented on a diff in pull request #41498: [SPARK-44001][Protobuf] spark protobuf: handle well known wrapper types

2023-06-08 Thread via GitHub
justaparth commented on code in PR #41498: URL: https://github.com/apache/spark/pull/41498#discussion_r1223202896 ## connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/ProtobufDeserializer.scala: ## @@ -247,12 +247,86 @@ private[sql] class ProtobufDeserializer(

[GitHub] [spark] justaparth commented on a diff in pull request #41498: [SPARK-44001][Protobuf] spark protobuf: handle well known wrapper types

2023-06-08 Thread via GitHub
justaparth commented on code in PR #41498: URL: https://github.com/apache/spark/pull/41498#discussion_r1223202896 ## connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/ProtobufDeserializer.scala: ## @@ -247,12 +247,86 @@ private[sql] class ProtobufDeserializer(

[GitHub] [spark] justaparth commented on a diff in pull request #41498: [SPARK-44001][Protobuf] spark protobuf: handle well known wrapper types

2023-06-08 Thread via GitHub
justaparth commented on code in PR #41498: URL: https://github.com/apache/spark/pull/41498#discussion_r1223192039 ## connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/ProtobufDeserializer.scala: ## @@ -247,12 +247,86 @@ private[sql] class ProtobufDeserializer(

[GitHub] [spark] justaparth commented on a diff in pull request #41498: [SPARK-44001][Protobuf] spark protobuf: handle well known wrapper types

2023-06-08 Thread via GitHub
justaparth commented on code in PR #41498: URL: https://github.com/apache/spark/pull/41498#discussion_r1223192039 ## connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/ProtobufDeserializer.scala: ## @@ -247,12 +247,86 @@ private[sql] class ProtobufDeserializer(

[GitHub] [spark] justaparth commented on a diff in pull request #41498: [SPARK-44001][Protobuf] spark protobuf: handle well known wrapper types

2023-06-08 Thread via GitHub
justaparth commented on code in PR #41498: URL: https://github.com/apache/spark/pull/41498#discussion_r1223192039 ## connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/ProtobufDeserializer.scala: ## @@ -247,12 +247,86 @@ private[sql] class ProtobufDeserializer(

[GitHub] [spark] justaparth commented on a diff in pull request #41498: [SPARK-44001][Protobuf] spark protobuf: handle well known wrapper types

2023-06-07 Thread via GitHub
justaparth commented on code in PR #41498: URL: https://github.com/apache/spark/pull/41498#discussion_r148127 ## connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/ProtobufDeserializer.scala: ## @@ -247,12 +247,86 @@ private[sql] class ProtobufDeserializer(

[GitHub] [spark] justaparth commented on a diff in pull request #41498: [SPARK-44001][Protobuf] spark protobuf: handle well known wrapper types

2023-06-07 Thread via GitHub
justaparth commented on code in PR #41498: URL: https://github.com/apache/spark/pull/41498#discussion_r148127 ## connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/ProtobufDeserializer.scala: ## @@ -247,12 +247,86 @@ private[sql] class ProtobufDeserializer(

[GitHub] [spark] justaparth commented on a diff in pull request #41498: [SPARK-44001][Protobuf] spark protobuf: handle well known wrapper types

2023-06-07 Thread via GitHub
justaparth commented on code in PR #41498: URL: https://github.com/apache/spark/pull/41498#discussion_r1222183033 ## connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/ProtobufDeserializer.scala: ## @@ -247,12 +247,86 @@ private[sql] class ProtobufDeserializer(