Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13991
Here it is for branch-2.0.
https://github.com/apache/spark/pull/14131
---
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 petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13991
@cloud-fan thanks for merging!
@yhuai I think the degree to which we want to add more tests also depend on
how much we trust the library we are using. XPath (with Query) is almost as
co
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13991
Thanks, merging to master! This doesn't merge clearly to 2.0, @petermaxlee
can you submit a new PR against 2.0? thanks!
---
If your project is set up for it, you can reply to this email and have
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/13991
OK. Thanks. Then, it will be good to add more tests for cases that are not
covered by those hive tests.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13991
BTW I have to say Hive's test coverage in this area is very spotty, so I
don't actually think it's great to follow, but I used those.
---
If your project is set up for it, you can reply to thi
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13991
Actually I created the unit tests based on those.
---
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 t
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/13991
As a follow-up task. Can you take a look at the following query files and
add useful tests in your test? Thanks.
```
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/describe_
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62070/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #62070 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62070/consoleFull)**
for PR 13991 at commit
[`0c60d87`](https://github.com/apache/spark/commit/
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13991
I guess whatever generates that message is buggy?
---
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 t
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13991
The latest result is `This patch does not merge cleanly.`, I just wanna
double check it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13991
@cloud-fan Jenkins already ran twice successfully before.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #62070 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62070/consoleFull)**
for PR 13991 at commit
[`0c60d87`](https://github.com/apache/spark/commit/0
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13991
LGTM, pending jenkins
---
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/13991
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #3175 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3175/consoleFull)**
for PR 13991 at commit
[`0c60d87`](https://github.com/apache/spark/commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #3175 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3175/consoleFull)**
for PR 13991 at commit
[`0c60d87`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62000/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #62000 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62000/consoleFull)**
for PR 13991 at commit
[`0c60d87`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #62000 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62000/consoleFull)**
for PR 13991 at commit
[`0c60d87`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61992/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61992 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61992/consoleFull)**
for PR 13991 at commit
[`fecc293`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61992 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61992/consoleFull)**
for PR 13991 at commit
[`fecc293`](https://github.com/apache/spark/commit/f
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13991
@hvanhovell can you take a look at this?
---
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 featu
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61975/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61975 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61975/consoleFull)**
for PR 13991 at commit
[`24deac9`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61967/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61967 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61967/consoleFull)**
for PR 13991 at commit
[`a86bc04`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61975 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61975/consoleFull)**
for PR 13991 at commit
[`24deac9`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61958/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61958 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61958/consoleFull)**
for PR 13991 at commit
[`48311bb`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61957/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61957 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61957/consoleFull)**
for PR 13991 at commit
[`d7d5f8d`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61967 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61967/consoleFull)**
for PR 13991 at commit
[`a86bc04`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61958 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61958/consoleFull)**
for PR 13991 at commit
[`48311bb`](https://github.com/apache/spark/commit/4
42 matches
Mail list logo
