[GitHub] spark pull request #18292: [SPARK-21083][SQL] Do not remove correct column s...

2017-07-08 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18292#discussion_r126276259 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -40,10 +39,10 @@ case class AnalyzeTableCommand(

[GitHub] spark pull request #18292: [SPARK-21083][SQL] Do not remove correct column s...

2017-07-08 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18292#discussion_r126276240 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala --- @@ -42,17 +42,20 @@ case class AnalyzeColumnCommand(

[GitHub] spark pull request #18292: [SPARK-21083][SQL] Do not remove correct column s...

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/18292#discussion_r126275492 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -40,10 +39,10 @@ case class

[GitHub] spark pull request #18292: [SPARK-21083][SQL] Do not remove correct column s...

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/18292#discussion_r126275485 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala --- @@ -42,17 +42,20 @@ case class

[GitHub] spark pull request #18292: [SPARK-21083][SQL] Do not remove correct column s...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18292#discussion_r126274515 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala --- @@ -42,17 +42,20 @@ case class