Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15568
thanks for the review, 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
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15568
lgtm, thanks for adding the 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15568
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/15568
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67444/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15568
**[Test build #67444 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67444/consoleFull)**
for PR 15568 at commit
[`c4a906f`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15568
**[Test build #67444 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67444/consoleFull)**
for PR 15568 at commit
[`c4a906f`](https://github.com/apache/spark/commit/c
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15568
@mallman , item 4 is a potential problem in the future. The current
workflow is, we get the `MetastoreRelation` via
`HiveMetastoreCatalog.lookupRelation`, which always lower case the database and
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15568
LGTM, but had a question on why we need to override equals().
---
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 h
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/15568
@cloud-fan, I don't know what you mean in item 4 about
`SessionCatalog.listPartitionsByFilter` handling case-sensitivity. What
case-sensitivity issue are you referring to, and does this PR handle it
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15568
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/15568
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67266/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15568
**[Test build #67266 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67266/consoleFull)**
for PR 15568 at commit
[`c98c4f6`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15568
**[Test build #67266 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67266/consoleFull)**
for PR 15568 at commit
[`c98c4f6`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15568
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67265/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15568
**[Test build #67265 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67265/consoleFull)**
for PR 15568 at commit
[`bd808af`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15568
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15568
cc @ericl @mallman @yhuai
---
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 wis
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15568
**[Test build #67265 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67265/consoleFull)**
for PR 15568 at commit
[`bd808af`](https://github.com/apache/spark/commit/b
18 matches
Mail list logo
