Re: [PR] [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession [spark]

2024-04-16 Thread via GitHub
zhengruifeng closed pull request #46075: [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession URL: https://github.com/apache/spark/pull/46075 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession [spark]

2024-04-16 Thread via GitHub
zhengruifeng commented on PR #46075: URL: https://github.com/apache/spark/pull/46075#issuecomment-2060071663 merged to master -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [PR] [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession [spark]

2024-04-16 Thread via GitHub
tomvanbussel commented on code in PR #46075: URL: https://github.com/apache/spark/pull/46075#discussion_r1567180994 ## connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala: ## @@ -586,9 +587,13 @@ class SparkSession private[sql] ( @DeveloperAp

Re: [PR] [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession [spark]

2024-04-16 Thread via GitHub
tomvanbussel commented on code in PR #46075: URL: https://github.com/apache/spark/pull/46075#discussion_r1566924760 ## connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala: ## @@ -586,9 +587,13 @@ class SparkSession private[sql] ( @DeveloperAp

Re: [PR] [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession [spark]

2024-04-16 Thread via GitHub
vicennial commented on code in PR #46075: URL: https://github.com/apache/spark/pull/46075#discussion_r1566908139 ## connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala: ## @@ -586,9 +587,13 @@ class SparkSession private[sql] ( @DeveloperApi

Re: [PR] [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession [spark]

2024-04-16 Thread via GitHub
vicennial commented on code in PR #46075: URL: https://github.com/apache/spark/pull/46075#discussion_r1566929334 ## connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala: ## @@ -586,9 +587,13 @@ class SparkSession private[sql] ( @DeveloperApi

Re: [PR] [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession [spark]

2024-04-16 Thread via GitHub
vicennial commented on code in PR #46075: URL: https://github.com/apache/spark/pull/46075#discussion_r1566908139 ## connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala: ## @@ -586,9 +587,13 @@ class SparkSession private[sql] ( @DeveloperApi

Re: [PR] [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession [spark]

2024-04-16 Thread via GitHub
tomvanbussel commented on code in PR #46075: URL: https://github.com/apache/spark/pull/46075#discussion_r1566878417 ## connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala: ## @@ -586,9 +587,13 @@ class SparkSession private[sql] ( @DeveloperAp

[PR] [SPARK-47868][CONNECT] Fix recursion limit error in SparkConnectPlanner and SparkSession [spark]

2024-04-16 Thread via GitHub
tomvanbussel opened a new pull request, #46075: URL: https://github.com/apache/spark/pull/46075 ### What changes were proposed in this pull request? This PR adds a helper function to `ProtoUtils` that calls a proto parser on a byte array with an increased recursion limit. This