Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@srowen Fixed the pending comments. Kindly recheck. Thanks
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95030/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #95030 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95030/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #95030 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95030/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94970/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94964/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94970 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94970/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94964 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94964/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94970 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94970/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94966/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94966 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94966/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94966 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94966/testReport)**
for PR 20611 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@srowen got your point, i will update
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20611
@sujith71955 we definitely shouldn't expose it to end users, but right now
it can be private until there's a reason for it to be less private to the rest
of Spark, I think.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94964 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94964/testReport)**
for PR 20611 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@srowen
Make this method private -- can be right?
This is more like a Util method where any feature deals with file system
can use this method to form a path instance without
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #4280 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4280/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #4280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4280/testReport)**
for PR 20611 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
Working fine with latest code. Thanks !!!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
Did some testing in my cluster with updated code for verifying the load
command with hdfs paths, please find the test results. Local path testing is
already covered in my UT.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94796/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94796 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94796/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94793/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94793 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94793/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94796 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94796/testReport)**
for PR 20611 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94793 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94793/testReport)**
for PR 20611 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94787 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94787/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94787/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94787 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94787/testReport)**
for PR 20611 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
Hi All, can we have a re-look into this PR and let me know whether is it
looking fine. Thanks
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94422/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94422 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94422/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94422 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94422/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94416/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94416 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94416/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94416 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94416/testReport)**
for PR 20611 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@gatorsmile i added the comment. Thanks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
Updated the PR by fixing the comment from sean. Hope i addressed all the
issues :)
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94359/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94359 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94359/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94359 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94359/testReport)**
for PR 20611 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94334/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94334 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94334/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94334 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94334/testReport)**
for PR 20611 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94300/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94300 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94300/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94300 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94300/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94298 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94298/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94298/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94298 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94298/testReport)**
for PR 20611 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
I will reiterate again, actually this PR was intended for fixing the issue
related to wildcard character issue in the hdfs file system scenarios , with
the current solution we are also able to
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
i think support wildcard is confusing term :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
No other changes in the load command behavior
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20611
> Yes there is a change in the behavior, As i mentioned above in
descriptions now we will be able to support wildcard even in the folder level
for local file systems. Previous versions will
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@gatorsmile Yes there is a change in the behavior, As i mentioned above in
descriptions now we will be able to support wildcard even in the folder level
for local file systems. Previous
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20611
Any behavior change?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
ok, i missed it, let me have a look. Thanks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
@sujith71955,
https://github.com/apache/spark/pull/20611#discussion_r202528097 was left after
my sign-off. I was expecting that's addressed.
---
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@srowen
The only possible objection I can see is that the behavior for paths that
contains a ? or * would now change :- There is no behavior change currently,
currently user cannot
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92978/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #92978 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92978/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #92978 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92978/testReport)**
for PR 20611 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
Look worth going ahead to me.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92968/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #92968 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92968/testReport)**
for PR 20611 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@srowen Thanks for the review. all comments has been addressed from my
side. let me know for any clarifications
---
-
To
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #92968 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92968/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #92966 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92966/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92966/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #92966 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92966/testReport)**
for PR 20611 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
This PR is open from many days, we need this fix in our version code also,
can someone please review and provide me suggestions if any.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92121/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #92121 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92121/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #92121 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92121/testReport)**
for PR 20611 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@HyukjinKwon Thanks for the re look.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
@kevinyu98, how does it look like? I think you're one of guys appropriate
to review this one since you took a look for the same issue.
---
1 - 100 of 165 matches
Mail list logo
