Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
I dont see I will be getting time to work on this. Will close the PR for
now and revisit in future.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14702
@tejasapatil gentle ping on the comments above.
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14702
Where is this file
https://github.com/apache/spark/blob/master/sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformationExec.scala
?
Your PR needs to remove it or
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14702
So far, we do not have any end-to-end test case for `ScriptTransformation`
without enabling Hive support. The test cases we have are all in hive/test:
[SQLQuerySuite.scala](https://github.com/apa
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14702
I might need more time to review this PR. Will keep posting my comments in
the next week.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14702
I will try to review it in the next few days. Thanks for working on it!
---
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 pr
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
can anyone please review this PR ?
---
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
enab
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72115/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #72115 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72115/testReport)**
for PR 14702 at commit
[`d9047f0`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #72115 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72115/testReport)**
for PR 14702 at commit
[`d9047f0`](https://github.com/apache/spark/commit/d9
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
Jenkins 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
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
jenkins retest please
test failure from build HiveSparkSubmitSuite `set
hive.metastore.warehouse.dir` is unrelated to the change
---
If your project is set up for it, you can reply to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71941/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #71941 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71941/testReport)**
for PR 14702 at commit
[`d9047f0`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #71941 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71941/testReport)**
for PR 14702 at commit
[`d9047f0`](https://github.com/apache/spark/commit/d9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71814/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #71814 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71814/testReport)**
for PR 14702 at commit
[`a6e9e39`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #71814 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71814/testReport)**
for PR 14702 at commit
[`a6e9e39`](https://github.com/apache/spark/commit/a6
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
can anyone please review this PR ?
---
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
enab
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71231/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #71231 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71231/testReport)**
for PR 14702 at commit
[`704e4a3`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #71231 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71231/testReport)**
for PR 14702 at commit
[`704e4a3`](https://github.com/apache/spark/commit/70
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71071/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #71071 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71071/testReport)**
for PR 14702 at commit
[`0436a17`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #71071 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71071/testReport)**
for PR 14702 at commit
[`0436a17`](https://github.com/apache/spark/commit/04
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70786/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #70786 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70786/testReport)**
for PR 14702 at commit
[`bb2a1ba`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #70786 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70786/testReport)**
for PR 14702 at commit
[`bb2a1ba`](https://github.com/apache/spark/commit/bb
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67040/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #67040 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67040/consoleFull)**
for PR 14702 at commit
[`c7741f9`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #67040 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67040/consoleFull)**
for PR 14702 at commit
[`c7741f9`](https://github.com/apache/spark/commit/c
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
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 s
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
@rxin : Yes. I think I know why its happening and will get back with a fix
over weekend.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14702
@tejasapatil looks like there is a legitimate failing 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 hav
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66723/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #66723 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66723/consoleFull)**
for PR 14702 at commit
[`c7741f9`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #66723 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66723/consoleFull)**
for PR 14702 at commit
[`c7741f9`](https://github.com/apache/spark/commit/c
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14702
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 tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
jenkins test this please.
Failed test from earlier run was in KafkaSourceStressSuite which I don't
see being related to this PR.
---
If your project is set up for it, you can reply to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66689/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #66689 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66689/consoleFull)**
for PR 14702 at commit
[`c7741f9`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #66689 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66689/consoleFull)**
for PR 14702 at commit
[`c7741f9`](https://github.com/apache/spark/commit/c
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
Jenkins test 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 wi
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14702
@tejasapatil I was checking with @hvanhovell. We should merge this one
soon. Mind bringing it up to date?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14702
@tejasapatil I'll take another pass tomorrow (CET).
---
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 tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
ping !!
---
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 th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65127/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #65127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65127/consoleFull)**
for PR 14702 at commit
[`f5256dd`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #65127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65127/consoleFull)**
for PR 14702 at commit
[`f5256dd`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65011/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #65011 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65011/consoleFull)**
for PR 14702 at commit
[`9afbd5e`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #65011 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65011/consoleFull)**
for PR 14702 at commit
[`9afbd5e`](https://github.com/apache/spark/commit/9
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14702
This looks reasonable.
cc @hvanhovell to take a look.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64321/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #64321 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64321/consoleFull)**
for PR 14702 at commit
[`9afbd5e`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #64321 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64321/consoleFull)**
for PR 14702 at commit
[`9afbd5e`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #64318 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64318/consoleFull)**
for PR 14702 at commit
[`3e97ec3`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64318/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #64318 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64318/consoleFull)**
for PR 14702 at commit
[`3e97ec3`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64313/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #64313 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64313/consoleFull)**
for PR 14702 at commit
[`9863c7d`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #64313 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64313/consoleFull)**
for PR 14702 at commit
[`9863c7d`](https://github.com/apache/spark/commit/9
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
@rxin : I have updated the description to include more info on changes done
and future todos
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14702
Can you update the description to say more about what this pr includes, and
what future todos are?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
cc @rxin : who would be the best person to review this PR ?
---
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 n
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63995/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
**[Test build #63995 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63995/consoleFull)**
for PR 14702 at commit
[`9dde09d`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #63995 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63995/consoleFull)**
for PR 14702 at commit
[`9dde09d`](https://github.com/apache/spark/commit/9
83 matches
Mail list logo