[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-27 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r415701513 ## File path: sql/core/src/main/scala/org/apache/spark/sql/Column.scala ## @@ -828,11 +828,12 @@ class Column(val expr: Expression) extends Logging {

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-27 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r415676504 ## File path: sql/core/src/main/scala/org/apache/spark/sql/Column.scala ## @@ -828,11 +828,12 @@ class Column(val expr: Expression) extends Logging {

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-26 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r415525400 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -426,7 +426,10 @@ case class In(value: Express

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-26 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r415521493 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -426,7 +426,10 @@ case class In(value: Express

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-26 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r415391444 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -426,7 +426,10 @@ case class In(value: Express

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-26 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r415256811 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -426,7 +426,10 @@ case class In(value: Express

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-25 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r415032708 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -519,7 +519,9 @@ case class InSet(child: Expre

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-25 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r415028923 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -519,7 +519,9 @@ case class InSet(child: Expre

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-24 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r414813908 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -519,7 +519,9 @@ case class InSet(child: Expre

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-24 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r414696382 ## File path: sql/core/src/main/scala/org/apache/spark/sql/Column.scala ## @@ -828,11 +828,11 @@ class Column(val expr: Expression) extends Logging {

[GitHub] [spark] MaxGekk commented on a change in pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

2020-04-24 Thread GitBox
MaxGekk commented on a change in pull request #28328: URL: https://github.com/apache/spark/pull/28328#discussion_r414566719 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -519,7 +519,9 @@ case class InSet(child: Expre