Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22794
I am not sure what Hive 1.2 exposes, but we might have more, it needs to be
checked. Anyway, yes, those have to be overridden for sure. When I referred to
the current approach, I meant the one desc
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22794
@mgaido91 You are right. But may be we only override
`newExecuteStatementOperation`, `newGetSchemasOperation`,
`newGetTablesOperation`, `newGetTableTypesOperation`,`newGetColumnsOperation`
and `new
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22794
@wangyum the point is that what I meant in the previous comment is that the
current approach is not good IMHO. Now you fixed it for the tables, but the
same is true for functions: currently you'd b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22794
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22794
**[Test build #98002 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98002/testReport)**
for PR 22794 at commit
[`80a8d21`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22794
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98002/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22794
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4464/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22794
**[Test build #98002 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98002/testReport)**
for PR 22794 at commit
[`80a8d21`](https://github.com/apache/spark/commit/80
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22794
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22794
cc @srinathshankar
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22794
**[Test build #97980 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97980/testReport)**
for PR 22794 at commit
[`e9a2a93`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22794
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97980/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22794
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22794
**[Test build #97980 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97980/testReport)**
for PR 22794 at commit
[`e9a2a93`](https://github.com/apache/spark/commit/e9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22794
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22794
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified//
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22794
cc @gatorsmile @srowen @dongjoon-hyun
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22794
Thanks @mgaido91 Changed to `sqlContext.sessionState.catalog` to obtain
table names now.
---
-
To unsubscribe, e-mail: reviews-u
18 matches
Mail list logo