mihailom-db commented on PR #46806:
URL: https://github.com/apache/spark/pull/46806#issuecomment-2139508692
@yaooqinn is this considered green? All CIs that were run passed, but most
of them were skipped.
--
This is an automated message from the Apache Git Service.
To respond to the
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1620668890
##
sql/core/benchmarks/CollationBenchmark-results.txt:
##
@@ -1,54 +1,79 @@
-OpenJDK 64-Bit Server VM 17.0.11+9-LTS on Linux 6.5.0-1018-azure
+OpenJDK 64-Bit Server VM
GideonPotok commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1620662883
##
sql/core/benchmarks/CollationBenchmark-results.txt:
##
@@ -1,54 +1,79 @@
-OpenJDK 64-Bit Server VM 17.0.11+9-LTS on Linux 6.5.0-1018-azure
+OpenJDK 64-Bit
GideonPotok commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1620663435
##
sql/core/benchmarks/CollationBenchmark-results.txt:
##
@@ -1,54 +1,79 @@
-OpenJDK 64-Bit Server VM 17.0.11+9-LTS on Linux 6.5.0-1018-azure
+OpenJDK 64-Bit
yaooqinn commented on PR #46806:
URL: https://github.com/apache/spark/pull/46806#issuecomment-2139427708
Thank you for the update. would you mind to raise another PR to target
master branch?Then, if CI is green here, we can merge that one with backport s
together.
--
This is an
mihailom-db commented on PR #46806:
URL: https://github.com/apache/spark/pull/46806#issuecomment-2139410598
@yaooqinn Switched base branch in new PR. Does this PR description seem
plausible?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
mihailom-db opened a new pull request, #46806:
URL: https://github.com/apache/spark/pull/46806
### What changes were proposed in this pull request?
Removal of stripMargin from the code in `DockerJDBCIntegrationV2Suite`.
### Why are the changes needed?
mihailom-db closed pull request #46803: [WIP][SPARK-48172][SQL] Fix escaping
issues in JDBCDialects
URL: https://github.com/apache/spark/pull/46803
--
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
yaooqinn commented on PR #46803:
URL: https://github.com/apache/spark/pull/46803#issuecomment-2139375639
Can you change the target branch to 3.5 and elaborate a bit more about the
issue and solution?
--
This is an automated message from the Apache Git Service.
To respond to the message,
mihailom-db opened a new pull request, #46803:
URL: https://github.com/apache/spark/pull/46803
### What changes were proposed in this pull request?
Fix for daily job failure on 3.4 branch.
### Why are the changes needed?
Tests failing.
### Does this PR introduce
mihailom-db opened a new pull request, #46801:
URL: https://github.com/apache/spark/pull/46801
### What changes were proposed in this pull request?
Addition of a test.
### Why are the changes needed?
Collations introduced a lot of changes to many functions and this test aims
uros-db opened a new pull request, #46788:
URL: https://github.com/apache/spark/pull/46788
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How
uros-db commented on PR #46772:
URL: https://github.com/apache/spark/pull/46772#issuecomment-2136550324
As noted in one of the resolved comments - there's no loss of coverage.
However, some tests have been (temporarily) removed because `StringTrim` no
longer supports UNICODE collation
uros-db commented on code in PR #46772:
URL: https://github.com/apache/spark/pull/46772#discussion_r1618200968
##
common/unsafe/src/test/java/org/apache/spark/unsafe/types/CollationSupportSuite.java:
##
@@ -994,20 +996,6 @@ public void testStringTrim() throws SparkException {
riyaverm-db commented on PR #46779:
URL: https://github.com/apache/spark/pull/46779#issuecomment-2136209502
@chaoqin-li1123
--
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.
mkaravel commented on code in PR #46772:
URL: https://github.com/apache/spark/pull/46772#discussion_r1617945708
##
common/unsafe/src/test/scala/org/apache/spark/unsafe/types/CollationFactorySuite.scala:
##
@@ -48,7 +48,7 @@ class CollationFactorySuite extends AnyFunSuite with
pkotikalapudi commented on PR #42352:
URL: https://github.com/apache/spark/pull/42352#issuecomment-2135858753
> hey guys, which version will this land in?
We have to get reviews and approvals from PMC members and our Sheperd
(@HeartSaVioR ) before set a timeline on when it can be
uros-db commented on PR #46597:
URL: https://github.com/apache/spark/pull/46597#issuecomment-2134820952
We can leave `PandasMode` for a separate PR, but we'll definitely need to
take care of it at one point
now that you've explored various options and finished the `groupMapReduce`
uros-db commented on PR #46772:
URL: https://github.com/apache/spark/pull/46772#issuecomment-2134806618
another note for future reference: as per [ICU
docs](https://unicode-org.github.io/icu/userguide/collation/concepts.html),
we've decided to stick with **TERTIARY** collation strength for
uros-db commented on code in PR #46772:
URL: https://github.com/apache/spark/pull/46772#discussion_r1616816500
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationFactory.java:
##
@@ -245,12 +245,12 @@ public CollationIdentifier identifier() {
uros-db opened a new pull request, #46772:
URL: https://github.com/apache/spark/pull/46772
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How
uros-db commented on code in PR #46720:
URL: https://github.com/apache/spark/pull/46720#discussion_r1615951326
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -135,22 +135,90 @@ public static UTF8String
GideonPotok commented on PR #46597:
URL: https://github.com/apache/spark/pull/46597#issuecomment-2132241154
@uros-db ?
--
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
stym06 commented on PR #42352:
URL: https://github.com/apache/spark/pull/42352#issuecomment-2132173624
hey guys, which version will this land in?
--
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
GideonPotok closed pull request #46404: [WIP][SPARK-47353][SQL] Enable
collation support for the Mode expression using a scala TreeMap (RB Tree)
URL: https://github.com/apache/spark/pull/46404
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
GideonPotok commented on PR #46597:
URL: https://github.com/apache/spark/pull/46597#issuecomment-2129923419
@uros-db I forgot but should I add collation support to
`org.apache.spark.sql.catalyst.expressions.aggregate.PandasMode`?
The only difference will be
1. Support for null
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1613363048
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -74,16 +90,29 @@ case class Mode(
if (buffer.isEmpty) {
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1613361676
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -74,16 +90,29 @@ case class Mode(
if (buffer.isEmpty) {
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1613358004
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -74,16 +90,29 @@ case class Mode(
if (buffer.isEmpty) {
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1613358004
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -74,16 +90,29 @@ case class Mode(
if (buffer.isEmpty) {
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1613351378
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -48,6 +49,21 @@ case class Mode(
override def inputTypes:
uros-db opened a new pull request, #46732:
URL: https://github.com/apache/spark/pull/46732
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How
uros-db commented on code in PR #46720:
URL: https://github.com/apache/spark/pull/46720#discussion_r1612994451
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationSupport.java:
##
@@ -208,87 +208,99 @@ public static boolean execICU(final UTF8String l,
uros-db commented on code in PR #46720:
URL: https://github.com/apache/spark/pull/46720#discussion_r1612993851
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationSupport.java:
##
@@ -208,87 +208,99 @@ public static boolean execICU(final UTF8String l,
uros-db commented on code in PR #46720:
URL: https://github.com/apache/spark/pull/46720#discussion_r1612949455
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -135,22 +135,90 @@ public static UTF8String
uros-db commented on code in PR #46720:
URL: https://github.com/apache/spark/pull/46720#discussion_r1612937162
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -135,22 +135,90 @@ public static UTF8String
uros-db commented on code in PR #46720:
URL: https://github.com/apache/spark/pull/46720#discussion_r1612936409
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationSupport.java:
##
@@ -208,87 +208,99 @@ public static boolean execICU(final UTF8String l,
uros-db commented on code in PR #46720:
URL: https://github.com/apache/spark/pull/46720#discussion_r1612933590
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -135,22 +135,90 @@ public static UTF8String
mkaravel commented on code in PR #46720:
URL: https://github.com/apache/spark/pull/46720#discussion_r1612654646
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -135,22 +135,90 @@ public static UTF8String
uros-db opened a new pull request, #46722:
URL: https://github.com/apache/spark/pull/46722
### What changes were proposed in this pull request?
Enable collation support for hash join on complex types.
- Logical plan is rewritten in analysis to (recursively) replace all
GideonPotok closed pull request #46488: [WIP][SPARK-47353][SQL][Prototype of
alternative algorithm] Enable collation support for the Mode expression using
multiple experimental approaches
URL: https://github.com/apache/spark/pull/46488
--
This is an automated message from the Apache Git
jiwen624 commented on code in PR #46714:
URL: https://github.com/apache/spark/pull/46714#discussion_r1611925610
##
sql/core/src/test/scala/org/apache/spark/sql/execution/metric/SQLMetricsSuite.scala:
##
@@ -837,7 +837,8 @@ class SQLMetricsSuite extends SharedSparkSession with
jiwen624 opened a new pull request, #46714:
URL: https://github.com/apache/spark/pull/46714
### What changes were proposed in this pull request?
For FileFormatDataWriter we currently record metrics of "task commit time"
and "job commit time" in
sunchao opened a new pull request, #46709:
URL: https://github.com/apache/spark/pull/46709
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How
GideonPotok commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1610561627
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -48,6 +49,22 @@ case class Mode(
override def inputTypes:
GideonPotok commented on PR #46597:
URL: https://github.com/apache/spark/pull/46597#issuecomment-2123419758
@uros-db I have made changes for all but your latest suggestion (re
whitelists -- will add that soon)
--
This is an automated message from the Apache Git Service.
To respond to the
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1608866089
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -48,6 +49,22 @@ case class Mode(
override def inputTypes:
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1608865653
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -48,6 +49,22 @@ case class Mode(
override def inputTypes:
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1608247812
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -74,18 +102,56 @@ case class Mode(
if (buffer.isEmpty) {
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1608240127
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -48,6 +49,33 @@ case class Mode(
override def inputTypes:
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1608242763
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -48,6 +49,33 @@ case class Mode(
override def inputTypes:
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1608240127
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -48,6 +49,33 @@ case class Mode(
override def inputTypes:
uros-db commented on code in PR #46597:
URL: https://github.com/apache/spark/pull/46597#discussion_r1608236375
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -48,6 +49,33 @@ case class Mode(
override def inputTypes:
uros-db opened a new pull request, #46682:
URL: https://github.com/apache/spark/pull/46682
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How
mkaravel commented on PR #46589:
URL: https://github.com/apache/spark/pull/46589#issuecomment-2121389075
Please update the PR description.
--
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
mkaravel commented on PR #46511:
URL: https://github.com/apache/spark/pull/46511#issuecomment-2121384473
Please fill in the PR description.
--
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
GideonPotok commented on PR #46597:
URL: https://github.com/apache/spark/pull/46597#issuecomment-2121332325
@uros-db I agree that we should avoid auxiliary structures. And I don't see
a good way to move the changes to implementation of `merge` and `update`
without keeping an auxiliary
GideonPotok closed pull request #46526: [WIP][SPARK-47353][SQL] Enable
collation support for the Mode expression using GroupMapReduce
URL: https://github.com/apache/spark/pull/46526
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
ericm-db opened a new pull request, #46644:
URL: https://github.com/apache/spark/pull/46644
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How
nchammas commented on code in PR #45340:
URL: https://github.com/apache/spark/pull/45340#discussion_r1605122743
##
docs/spark-connect-extending.md:
##
@@ -0,0 +1,248 @@
+---
+layout: global
+title: Extending Spark Connect with Custom Functionality
+license: |
+ Licensed to the
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1604485097
##
common/unsafe/src/test/java/org/apache/spark/unsafe/types/CollationSupportSuite.java:
##
@@ -709,12 +774,24 @@ public void testLocate() throws SparkException {
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1604482878
##
common/unsafe/src/test/java/org/apache/spark/unsafe/types/CollationSupportSuite.java:
##
@@ -709,12 +774,24 @@ public void testLocate() throws SparkException {
mkaravel commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1603739103
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -34,6 +34,143 @@
* Utility class for collation-aware
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1604439268
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -34,6 +34,143 @@
* Utility class for collation-aware
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1604437739
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -34,6 +34,143 @@
* Utility class for collation-aware
uros-db commented on code in PR #46599:
URL: https://github.com/apache/spark/pull/46599#discussion_r1604352238
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/RewriteCollationJoin.scala:
##
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software
uros-db commented on code in PR #46599:
URL: https://github.com/apache/spark/pull/46599#discussion_r1604346112
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/RewriteCollationJoin.scala:
##
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software
github-actions[bot] closed pull request #45001: [WIP][Spark 44646] Reduce usage
of log4j core
URL: https://github.com/apache/spark/pull/45001
--
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
mkaravel commented on code in PR #46589:
URL: https://github.com/apache/spark/pull/46589#discussion_r1603752983
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -278,47 +431,29 @@ public static UTF8String
mkaravel commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1603716669
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -34,6 +34,143 @@
* Utility class for collation-aware
cloud-fan commented on code in PR #46599:
URL: https://github.com/apache/spark/pull/46599#discussion_r1603420522
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/RewriteCollationJoin.scala:
##
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software
dbatomic commented on code in PR #46599:
URL: https://github.com/apache/spark/pull/46599#discussion_r1603381135
##
sql/core/src/test/scala/org/apache/spark/sql/CollationSuite.scala:
##
@@ -1051,6 +1052,153 @@ class CollationSuite extends DatasourceV2SQLBase with
dbatomic commented on code in PR #46599:
URL: https://github.com/apache/spark/pull/46599#discussion_r1603377917
##
sql/core/src/test/scala/org/apache/spark/sql/CollationSuite.scala:
##
@@ -784,19 +785,19 @@ class CollationSuite extends DatasourceV2SQLBase with
dbatomic commented on code in PR #46599:
URL: https://github.com/apache/spark/pull/46599#discussion_r1603376505
##
sql/core/src/test/scala/org/apache/spark/sql/CollationSuite.scala:
##
@@ -784,19 +785,19 @@ class CollationSuite extends DatasourceV2SQLBase with
dbatomic commented on code in PR #46599:
URL: https://github.com/apache/spark/pull/46599#discussion_r1603374449
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala:
##
@@ -397,7 +398,11 @@ trait JoinSelectionHelper extends Logging {
protected
dbatomic commented on code in PR #46599:
URL: https://github.com/apache/spark/pull/46599#discussion_r1603367540
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CollationKey.scala:
##
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the Apache Software Foundation
jiwen624 opened a new pull request, #46604:
URL: https://github.com/apache/spark/pull/46604
### What changes were proposed in this pull request?
Working on it...
### Why are the changes needed?
As mentioned in the Jira ticket:
https://issues.apache.org/jira/browse/SPARK-48298
github-actions[bot] commented on PR #45001:
URL: https://github.com/apache/spark/pull/45001#issuecomment-2113687236
We're closing this PR because it hasn't been updated in a while. This isn't
a judgement on the merit of the PR in any way. It's just a way of keeping the
PR queue manageable.
uros-db opened a new pull request, #46599:
URL: https://github.com/apache/spark/pull/46599
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How
GideonPotok opened a new pull request, #46597:
URL: https://github.com/apache/spark/pull/46597
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
###
uros-db opened a new pull request, #46589:
URL: https://github.com/apache/spark/pull/46589
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How
GideonPotok commented on code in PR #46526:
URL: https://github.com/apache/spark/pull/46526#discussion_r1600668920
##
sql/core/src/test/scala/org/apache/spark/sql/CollationStringExpressionsSuite.scala:
##
@@ -22,7 +22,7 @@ import
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1600495650
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -183,6 +204,19 @@ public static int findInSet(final UTF8String
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1600495650
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -183,6 +204,19 @@ public static int findInSet(final UTF8String
uros-db commented on code in PR #46526:
URL: https://github.com/apache/spark/pull/46526#discussion_r1600471419
##
sql/core/benchmarks/CollationBenchmark-jdk21-results.txt:
##
Review Comment:
all things considered, I would say proceed with this approach - clean
everything
uros-db commented on code in PR #46526:
URL: https://github.com/apache/spark/pull/46526#discussion_r1600468023
##
sql/core/src/test/scala/org/apache/spark/sql/CollationStringExpressionsSuite.scala:
##
Review Comment:
"mode" is not a StringExpression
let's move the
uros-db commented on code in PR #46526:
URL: https://github.com/apache/spark/pull/46526#discussion_r1600463245
##
sql/core/benchmarks/CollationBenchmark-jdk21-results.txt:
##
Review Comment:
apropos altering the benchmark to yield better results for this particular
uros-db commented on code in PR #46526:
URL: https://github.com/apache/spark/pull/46526#discussion_r1600457296
##
sql/core/benchmarks/CollationBenchmark-jdk21-results.txt:
##
Review Comment:
apropos going through lower first, we need to be careful so as not to
destroy the
GideonPotok commented on code in PR #46526:
URL: https://github.com/apache/spark/pull/46526#discussion_r1600041876
##
sql/core/benchmarks/CollationBenchmark-jdk21-results.txt:
##
Review Comment:
0. Note, by the way that because we are relying on supportsBinaryEquality,
mkaravel commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1600292083
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -183,6 +204,19 @@ public static int findInSet(final UTF8String
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1599873181
##
common/unsafe/src/test/java/org/apache/spark/unsafe/types/CollationSupportSuite.java:
##
@@ -102,20 +102,30 @@ public void testContains() throws SparkException {
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1599772157
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -183,6 +204,19 @@ public static int findInSet(final UTF8String
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1599736471
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -183,6 +204,19 @@ public static int findInSet(final UTF8String
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1599689228
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -34,6 +34,27 @@
* Utility class for collation-aware UTF8String
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1599693830
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -34,6 +34,27 @@
* Utility class for collation-aware UTF8String
uros-db commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1599689228
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -34,6 +34,27 @@
* Utility class for collation-aware UTF8String
mkaravel commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1599455730
##
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationAwareUTF8String.java:
##
@@ -34,6 +34,27 @@
* Utility class for collation-aware
mkaravel commented on code in PR #46511:
URL: https://github.com/apache/spark/pull/46511#discussion_r1599447728
##
common/unsafe/src/test/java/org/apache/spark/unsafe/types/CollationSupportSuite.java:
##
@@ -102,20 +102,30 @@ public void testContains() throws SparkException {
uros-db commented on code in PR #46526:
URL: https://github.com/apache/spark/pull/46526#discussion_r1598850667
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Mode.scala:
##
@@ -74,6 +78,18 @@ case class Mode(
if (buffer.isEmpty) {
1 - 100 of 760 matches
Mail list logo