Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/23108
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22141
@maropu Very sorry. I haven't had the time to come back to it. I have some
stuff on my plate. I will get to this after i am done. T
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/23108
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/23211
@wangyum Thanks.. Can you please tell me how you generate this ? Also, is
it possible to get runtimes of these queries to see if there are any
regressions
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22899
@gatorsmile Thanks a lot. I completely agree that we should try and combine
these two. I will continue to think about it
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/23211
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/23211
[SPARK-19712][SQL] Move PullupCorrelatedPredicates and
RewritePredicateSubquery after OptimizeSubqueries
Currently predicate subqueries (IN/EXISTS) are converted to Joins at the
end of
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/23170
Looks good to me. Thanks for fixing this @wangyum. I guess, i tried to trim
off the example to show the mutual exclusivity of these two parameters in
question
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r232571944
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Max.scala
---
@@ -57,3 +57,34 @@ case class Max(child
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22899
cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22899
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22788#discussion_r229797016
--- Diff:
sql/core/src/test/resources/sql-tests/results/columnresolution-negative.sql.out
---
@@ -161,7 +161,7 @@ SELECT db1.t1.i1 FROM t1, mydb2.t1
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22899
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22900
Oh...thank you very much @dongjoon-hyun
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22900#discussion_r229625779
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaDontFailOnDataLossSuite.scala
---
@@ -221,7 +221,7 @@ class
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22900
cc @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22900
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22895
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/21860
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22847
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22899
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22788#discussion_r229578738
--- Diff:
sql/core/src/test/resources/sql-tests/results/columnresolution-negative.sql.out
---
@@ -161,7 +161,7 @@ SELECT db1.t1.i1 FROM t1, mydb2.t1
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22788#discussion_r229577806
--- Diff:
sql/core/src/test/resources/sql-tests/results/columnresolution-negative.sql.out
---
@@ -161,7 +161,7 @@ SELECT db1.t1.i1 FROM t1, mydb2.t1
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22900
[SPARK-25618][SQL][TEST] Reduce time taken to execute
KafkaContinuousSourceStressForDontFailOnDataLossSuite
## What changes were proposed in this pull request?
In this test, i have reduced
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22899
cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22899
[SPARK-25573] Combine resolveExpression and resolve in the Analyzer
## What changes were proposed in this pull request?
Currently in the Analyzer, we have two methods 1) Resolve
2
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/17520
@gatorsmile Sure Sean.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22778#discussion_r229362181
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -171,10 +171,13 @@ abstract class Optimizer
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22778#discussion_r229181821
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -171,10 +171,13 @@ abstract class Optimizer
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22868#discussion_r229063489
--- Diff: docs/sql-migration-guide-hive-compatibility.md ---
@@ -53,7 +53,20 @@ Spark SQL supports the vast majority of Hive features,
such as
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22868#discussion_r229062732
--- Diff: docs/sql-migration-guide-hive-compatibility.md ---
@@ -51,6 +51,22 @@ Spark SQL supports the vast majority of Hive features,
such as
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22868#discussion_r229051205
--- Diff: docs/sql-migration-guide-hive-compatibility.md ---
@@ -53,7 +53,20 @@ Spark SQL supports the vast majority of Hive features,
such as
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22856
Thank you @srowen @dongjoon-hyun @mgaido91 @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22868#discussion_r228766091
--- Diff: docs/sql-migration-guide-hive-compatibility.md ---
@@ -51,6 +51,9 @@ Spark SQL supports the vast majority of Hive features,
such as
Github user dilipbiswal closed the pull request at:
https://github.com/apache/spark/pull/22797
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22809
Thanks a lot @cloud-fan @mgaido91
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dilipbiswal closed the pull request at:
https://github.com/apache/spark/pull/22047
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228725645
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/UnevaluableAggs.scala
---
@@ -0,0 +1,62
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228666789
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/UnevaluableAggs.scala
---
@@ -0,0 +1,62
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22856
[SPARK-25856][SQL][MINOR] Remove AverageLike and CountLike classes
## What changes were proposed in this pull request?
These two classes were added for regr_ expression support (SPARK-23907
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228644504
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/UnevaluableAggs.scala
---
@@ -0,0 +1,62
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228638310
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/UnevaluableAggs.scala
---
@@ -0,0 +1,62
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228636110
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala
---
@@ -282,6 +283,31 @@ trait RuntimeReplaceable
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228634827
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala
---
@@ -282,6 +283,31 @@ trait RuntimeReplaceable
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228634159
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/UnevaluableAggs.scala
---
@@ -0,0 +1,62
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22809
@cloud-fan @mgaido91 I have incorporated the comments. Could we please
check if things look okay now ?
---
-
To unsubscribe
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228239274
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Max.scala
---
@@ -57,3 +57,34 @@ case class Max(child
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228235105
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Max.scala
---
@@ -57,3 +57,34 @@ case class Max(child
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228229829
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Max.scala
---
@@ -57,3 +57,34 @@ case class Max(child
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228227624
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Max.scala
---
@@ -57,3 +57,34 @@ case class Max(child
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22821
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r228020366
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -727,4 +728,67 @@ class DataFrameAggregateSuite extends
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r227901365
--- Diff: sql/core/src/test/resources/sql-tests/inputs/group-by.sql ---
@@ -80,3 +80,69 @@ SELECT 1 FROM range(10) HAVING true;
SELECT 1 FROM range
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22809
@cloud-fan Yeah.. I have some tests in group-by.sql. Please take a look.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22809#discussion_r227881077
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/finishAnalysis.scala
---
@@ -38,6 +39,18 @@ object ReplaceExpressions
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22809
cc @cloud-fan @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22809
[SPARK-19851][SQL] Add support for EVERY and ANY (SOME) aggregates
## What changes were proposed in this pull request?
Implements Every, Some, Any aggregates in SQL. These new
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22797
@cloud-fan OK.. thanks a LOT.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22797
@cloud-fan Sure.. Let me give that a try. Thanks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22797
cc @cloud-fan @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22797
[SPARK-19851][SQL] Add support for EVERY and ANY (SOME) aggregates
## What changes were proposed in this pull request?
Implements Every, Some, Any aggregates in SQL. Logically these
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22047#discussion_r227054412
--- Diff: python/pyspark/sql/functions.py ---
@@ -403,6 +403,28 @@ def countDistinct(col, *cols):
return Column(jc)
+def every
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22772
Thank you @gatorsmile @HyukjinKwon @xuanyuanking
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22772
@HyukjinKwon Sure..
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22772
@HyukjinKwon Sure. If there are more changes planned in another PR, then it
can take these changes in. Since this went to 2.4, i thought there may be some
urgency to fix the links
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22772
cc @dongjoon-hyun @gatorsmile @xuanyuanking
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22772
[SPARK-24499][SQL][DOC][Followup] Fix some broken links
## What changes were proposed in this pull request?
Fix some broken links in the new document. I have clicked through all the
links
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22670
@srowen Thank you very much.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22047#discussion_r225700828
--- Diff: python/pyspark/sql/functions.py ---
@@ -403,6 +403,28 @@ def countDistinct(col, *cols):
return Column(jc)
+def every
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22670
@srowen OK, Let me look.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22670
@srowen Thanks. Did you mean, the test cases should extend a shared spark
context (SharedKafkaSparkContext) which would have this property set ?
Actually Sean, there are 3 suites in
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22671
Thanks a LOT @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22641
Thank you very much @srowen @fjh100456
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22671#discussion_r223819177
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -332,7 +332,9 @@ class KafkaSinkSuite
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22671#discussion_r223771001
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -332,7 +332,9 @@ class KafkaSinkSuite
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22641
@fjh100456 Yeah.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22671
cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22671
[SPARK-25615][SQL][TEST] Improve the test runtime of KafkaSinkSuite:
streaming write to non-existing topic
## What changes were proposed in this pull request?
Specify `kafka.max.block.ms
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22594
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22670
cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22670
[SPARK-25631][SPARK-25632][SQL][TEST] Improve the test runtime of
KafkaRDDSuite
## What changes were proposed in this pull request?
Set a reasonable poll timeout thats used while consuming
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22641
@srowen @gatorsmile
Let me see if we can target a low hanging fruit first :-). Given the same
input set is used for table and session codecs, we can skip when they both are
same value
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22641
@srowen
> I would vote against running tests that we think have any value randomly.
It's just the wrong way to solve problems, as much as it would be to simply run
90% of our tes
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22641
@srowen Thank you for your comments. Actually from a cursory look, i would
agree that it does not look that pretty. I also agree that it does look like we
are not testing as much as we used to
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/21732
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22500
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22448
@HyukjinKwon OK.. will do.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22641
@mgaido91
Thanks for your input.
I took another look at the testcase. Let me outline some of my
understandings first.
- The test validates the precedence rules in
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22648
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22047#discussion_r223171963
--- Diff: python/pyspark/sql/functions.py ---
@@ -403,6 +403,28 @@ def countDistinct(col, *cols):
return Column(jc)
+def every
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22638
Thanks a lot @gatorsmile @mgaido91
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22644
Thank you very much @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22644
cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22642
@dongjoon-hyun Ah.. got it.. thanks a lot.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22644
[SPARK-25626] Improve the test execution time of HiveClientSuites
## What changes were proposed in this pull request?
Improve the runtime by reducing the number of partitions created in the
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22642
@gengliangwang I have a question. What happens when we place this tag ? It
is skipped on regular runs ?
---
-
To
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22047
@cloud-fan please see my comment
[link](https://github.com/apache/spark/pull/22047#issuecomment-411536039). I
had tried to rewrite using max and min as suggested by Herman and Reynold in
the
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22641
@mgaido91 Trying to understand the concern regarding "working combination
and a non-working " comment. In my understanding, originally we were doing a
cross join between two sets of
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22641
@mgaido91 Could you please take a look ? Thank you.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22641
[SPARK-25611][SPARK-25622][SQL][TESTS] Improve test run time of
CompressionCodecSuite
## What changes were proposed in this pull request?
I am using the idea of @mgaido91 to pick the test
1 - 100 of 1260 matches
Mail list logo