[GitHub] [spark] LuciferYang commented on pull request #39682: [SPARK-42139][CORE][SQL] Handle null string values in SQLExecutionUIData/SparkPlanGraphWrapper/SQLPlanMetric

LuciferYang commented on PR #39682: URL: https://github.com/apache/spark/pull/39682#issuecomment-1399435222 This one GA passed -- 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

[GitHub] [spark] LuciferYang commented on pull request #39682: [SPARK-42139][CORE][SQL] Handle null string values in SQLExecutionUIData/SparkPlanGraphWrapper/SQLPlanMetric

LuciferYang commented on PR #39682: URL: https://github.com/apache/spark/pull/39682#issuecomment-1399397378 rebased -- 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 unsu

[GitHub] [spark] LuciferYang commented on pull request #39682: [SPARK-42139][CORE][SQL] Handle null string values in SQLExecutionUIData/SparkPlanGraphWrapper/SQLPlanMetric

LuciferYang commented on PR #39682: URL: https://github.com/apache/spark/pull/39682#issuecomment-1399169469 And there is another pr do similar work https://github.com/apache/spark/pull/39682 @mridulm -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [spark] LuciferYang commented on pull request #39682: [SPARK-42139][CORE][SQL] Handle null string values in SQLExecutionUIData/SparkPlanGraphWrapper/SQLPlanMetric

LuciferYang commented on PR #39682: URL: https://github.com/apache/spark/pull/39682#issuecomment-1399169247 For String type, `SomeProtoBufBuilder. setString (null)` will throw NPE, so in https://github.com/apache/spark/pull/39666 @gengliangwang propose: - checking nullability of all s

[GitHub] [spark] LuciferYang commented on pull request #39682: [SPARK-42139][CORE][SQL] Handle null string values in SQLExecutionUIData/SparkPlanGraphWrapper/SQLPlanMetric

LuciferYang commented on PR #39682: URL: https://github.com/apache/spark/pull/39682#issuecomment-1399167087 cc @gengliangwang -- 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