Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16811#discussion_r104861465
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/Word2VecSuite.scala ---
@@ -134,13 +134,20 @@ class Word2VecSuite extends SparkFunSuite with
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17204#discussion_r104861159
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/SQLTestUtils.scala ---
@@ -337,4 +339,15 @@ private[sql] object SQLTestUtils {
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17200#discussion_r104861095
--- Diff: python/pyspark/sql/types.py ---
@@ -189,7 +189,7 @@ def toInternal(self, dt):
if dt is not None:
seconds =
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17204
**[Test build #74189 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74189/testReport)**
for PR 17204 at commit
Github user witgo commented on the issue:
https://github.com/apache/spark/pull/17139
Added the multi-threaded code for serialization `TaskDescription` .
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17204
**[Test build #74188 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74188/testReport)**
for PR 17204 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17201
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74177/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17201
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17146
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17200#discussion_r104858211
--- Diff: python/pyspark/sql/types.py ---
@@ -189,7 +189,7 @@ def toInternal(self, dt):
if dt is not None:
seconds =
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17201
**[Test build #74177 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74177/testReport)**
for PR 17201 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16729
yea - I'm sorry if it was confusing - I was referring to `SparkR::tweedie`
(2 `:`), and not `SparkR:::tweedie` (3 `:`), which was why I wasn't sure that
could be done.
In the past
Github user JasonMWhite commented on a diff in the pull request:
https://github.com/apache/spark/pull/17200#discussion_r104856801
--- Diff: python/pyspark/sql/types.py ---
@@ -189,7 +189,7 @@ def toInternal(self, dt):
if dt is not None:
seconds =
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16933
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74179/
Test PASSed.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17096#discussion_r104856634
--- Diff: python/pyspark/ml/tests.py ---
@@ -320,6 +320,13 @@ def test_hasparam(self):
testParams = TestParams()
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16933
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16933
**[Test build #74179 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74179/testReport)**
for PR 16933 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17205
LGTM too
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user JasonMWhite commented on a diff in the pull request:
https://github.com/apache/spark/pull/17200#discussion_r104856310
--- Diff: python/pyspark/sql/types.py ---
@@ -189,7 +189,7 @@ def toInternal(self, dt):
if dt is not None:
seconds =
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17204
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74176/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17204
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17197
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74180/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17204
**[Test build #74176 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74176/testReport)**
for PR 17204 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17197
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17197
**[Test build #74180 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74180/testReport)**
for PR 17197 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16944
**[Test build #74187 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74187/testReport)**
for PR 16944 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17075
**[Test build #74186 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74186/testReport)**
for PR 17075 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17200#discussion_r104855668
--- Diff: python/pyspark/sql/types.py ---
@@ -189,7 +189,7 @@ def toInternal(self, dt):
if dt is not None:
seconds =
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16944
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17075
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16729
@felixcheung Could you take a look at this new fix when you get a chance?
Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15009
**[Test build #74185 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74185/testReport)**
for PR 15009 at commit
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17146#discussion_r104853968
--- Diff: python/pyspark/ml/tests.py ---
@@ -1223,6 +1223,26 @@ def test_apply_binary_term_freqs(self):
":
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/17146
This looks good to me. Thanks
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15363
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74175/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15363
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15363
**[Test build #74175 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74175/testReport)**
for PR 15363 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17186
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17186
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74174/
Test PASSed.
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17202#discussion_r104853064
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -576,6 +576,8 @@ class Dataset[T] private[sql](
val parsedDelay =
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17186
**[Test build #74174 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74174/testReport)**
for PR 17186 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15009
**[Test build #74184 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74184/testReport)**
for PR 15009 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17205
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17075
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74181/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17075
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17075
**[Test build #74181 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74181/testReport)**
for PR 17075 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17205
**[Test build #74183 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74183/testReport)**
for PR 17205 at commit
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/17184#discussion_r104851283
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -850,26 +850,27 @@ public UTF8String
GitHub user tejasapatil opened a pull request:
https://github.com/apache/spark/pull/17205
[SPARK-19843] [SQL] [Followup] Classdoc for `IntWrapper` and `LongWrapper`
## What changes were proposed in this pull request?
This is as per suggestion by @rxin at :
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17186
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17186
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74173/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17186
**[Test build #74173 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74173/testReport)**
for PR 17186 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17202
**[Test build #74182 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74182/testReport)**
for PR 17202 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17202
cc @zsxwing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user davies commented on the issue:
https://github.com/apache/spark/pull/16896
My bad, did not realized that, sorry.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user JasonMWhite commented on the issue:
https://github.com/apache/spark/pull/17200
Ah, test failed on Python 3.4 only. That makes some sense, I only tested
locally on 2.6, and there are changes with how Python 3 handles ints vs longs.
I'll dig in with Python 3.4 and see if I
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17202
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17202
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74172/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17202
**[Test build #74172 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74172/testReport)**
for PR 17202 at commit
Github user JasonMWhite commented on a diff in the pull request:
https://github.com/apache/spark/pull/17200#discussion_r104847039
--- Diff: python/pyspark/sql/tests.py ---
@@ -1435,6 +1435,12 @@ def test_time_with_timezone(self):
self.assertEqual(now, now1)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17001
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17001
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74170/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17001
**[Test build #74170 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74170/testReport)**
for PR 17001 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17200#discussion_r104845981
--- Diff: python/pyspark/sql/tests.py ---
@@ -1435,6 +1435,12 @@ def test_time_with_timezone(self):
self.assertEqual(now, now1)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17186
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17186
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74171/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17186
**[Test build #74171 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74171/testReport)**
for PR 17186 at commit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17184#discussion_r104845661
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -897,41 +898,52 @@ public long toLong() {
break;
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15180
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/17184#discussion_r104843817
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -897,41 +898,52 @@ public long toLong() {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17200
**[Test build #74178 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74178/testReport)**
for PR 17200 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17200
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74178/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17200
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user BryanCutler closed the pull request at:
https://github.com/apache/spark/pull/17193
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user BryanCutler closed the pull request at:
https://github.com/apache/spark/pull/17195
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/17195
Sure, no problem!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/17195
LGTM
Merging with branch-2.0
Thank you again!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/17193
LGTM
Merging with branch-2.1
Thank you!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17075
**[Test build #74181 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74181/testReport)**
for PR 17075 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17197
**[Test build #74180 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74180/testReport)**
for PR 17197 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/17197
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16811
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16811
Thanks! I made a follow-up JIRA for updating the Python API:
https://issues.apache.org/jira/browse/SPARK-19866
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17200
**[Test build #74178 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74178/testReport)**
for PR 17200 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17075
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16933
**[Test build #74179 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74179/testReport)**
for PR 16933 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17199
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17200
ok to test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16933
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16811
LGTM
Merging with master
Thank you!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17183
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17184#discussion_r104842090
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -897,41 +898,52 @@ public long toLong() {
break;
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/17199
Thanks! Merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/17183
Thanks! Merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17184
I believe IBM J9 actually improved this specific case (their JIT handles
tons of exceptions better). Oh well -- if only JIT is perfect.
---
If your project is set up for it, you can reply to this
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16883
Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/17184#discussion_r104841803
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -850,26 +850,27 @@ public UTF8String
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17184#discussion_r104841789
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -897,41 +898,52 @@ public long toLong() {
break;
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17184#discussion_r104841761
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -897,41 +898,52 @@ public long toLong() {
break;
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17184#discussion_r104841735
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -850,26 +850,27 @@ public UTF8String translate(Map
1 - 100 of 801 matches
Mail list logo