Re: [PR] [SPARK-45868][CONNECT] Make sure `spark.table` use the same parser with vanilla spark [spark]

2023-11-15 Thread via GitHub
dongjoon-hyun commented on PR #43741: URL: https://github.com/apache/spark/pull/43741#issuecomment-1813308124 Merged to master. Thank you, @zhengruifeng and all! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [SPARK-45868][CONNECT] Make sure `spark.table` use the same parser with vanilla spark [spark]

2023-11-15 Thread via GitHub
dongjoon-hyun closed pull request #43741: [SPARK-45868][CONNECT] Make sure `spark.table` use the same parser with vanilla spark URL: https://github.com/apache/spark/pull/43741 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] [SPARK-45868][CONNECT] Make sure `spark.table` use the same parser with vanilla spark [spark]

2023-11-14 Thread via GitHub
zhengruifeng commented on PR #43741: URL: https://github.com/apache/spark/pull/43741#issuecomment-1809718195 cc @dongjoon-hyun @HyukjinKwon would you mind taking another look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] [SPARK-45868][CONNECT] Make sure `spark.table` use the same parser with vanilla spark [spark]

2023-11-13 Thread via GitHub
zhengruifeng commented on PR #43741: URL: https://github.com/apache/spark/pull/43741#issuecomment-1809510311 since it dose cause some issues in productions, I think we need to also backport it to 3.5 -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] [SPARK-45868][CONNECT] Make sure `spark.table` use the same parser with vanilla spark [spark]

2023-11-13 Thread via GitHub
zhengruifeng commented on code in PR #43741: URL: https://github.com/apache/spark/pull/43741#discussion_r1391956660 ## connector/connect/server/src/main/scala/org/apache/spark/sql/connect/planner/SparkConnectPlanner.scala: ## @@ -1217,7 +1217,8 @@ class SparkConnectPlanner(

Re: [PR] [SPARK-45868][CONNECT] Make sure `spark.table` use the same parser with vanilla spark [spark]

2023-11-10 Thread via GitHub
zhengruifeng commented on code in PR #43741: URL: https://github.com/apache/spark/pull/43741#discussion_r1389908398 ## connector/connect/server/src/main/scala/org/apache/spark/sql/connect/planner/SparkConnectPlanner.scala: ## @@ -1217,7 +1217,8 @@ class SparkConnectPlanner(

Re: [PR] [SPARK-45868][CONNECT] Make sure `spark.table` use the same parser with vanilla spark [spark]

2023-11-09 Thread via GitHub
HyukjinKwon commented on code in PR #43741: URL: https://github.com/apache/spark/pull/43741#discussion_r1389039188 ## connector/connect/server/src/main/scala/org/apache/spark/sql/connect/planner/SparkConnectPlanner.scala: ## @@ -1217,7 +1217,8 @@ class SparkConnectPlanner(

Re: [PR] [SPARK-45868][CONNECT] Make sure `spark.table` use the same parser with vanilla spark [spark]

2023-11-09 Thread via GitHub
zhengruifeng commented on PR #43741: URL: https://github.com/apache/spark/pull/43741#issuecomment-1804690249 cc @ueshin -- 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. To