[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-27 Thread asfgit 
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/12721


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-27 Thread rxin 
Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-215232158
  
Merging in 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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-27 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-215207794
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57152/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-27 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-215207790
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-27 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-215207522
  
**[Test build #57152 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57152/consoleFull)**
 for PR 12721 at commit 
[`8a19b61`](https://github.com/apache/spark/commit/8a19b616f0b28ed01ca1c94807e942b39efb72ca).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds the following public classes _(experimental)_:
  * `abstract class ExternalCatalog `


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-27 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-215177688
  
**[Test build #57152 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57152/consoleFull)**
 for PR 12721 at commit 
[`8a19b61`](https://github.com/apache/spark/commit/8a19b616f0b28ed01ca1c94807e942b39efb72ca).


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-27 Thread andrewor14 
Github user andrewor14 commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-215170571
  
yeah, also doesn't compile. Let me fix that.


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-26 Thread rxin 
Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-214972806
  
I guess ... conflict now


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-26 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-214922257
  
**[Test build #57055 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57055/consoleFull)**
 for PR 12721 at commit 
[`42c858d`](https://github.com/apache/spark/commit/42c858d4e1b8ad5c49d3c65b218d38f5c0c3ebab).
 * This patch **fails to build**.
 * This patch merges cleanly.
 * This patch adds the following public classes _(experimental)_:
  * `abstract class ExternalCatalog `


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-26 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-214922261
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-26 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-214922264
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57055/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-26 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12721#issuecomment-214921697
  
**[Test build #57055 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57055/consoleFull)**
 for PR 12721 at commit 
[`42c858d`](https://github.com/apache/spark/commit/42c858d4e1b8ad5c49d3c65b218d38f5c0c3ebab).


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-26 Thread andrewor14 
GitHub user andrewor14 opened a pull request:

https://github.com/apache/spark/pull/12721

[SPARK-14940][SQL] Move ExternalCatalog to own file

## What changes were proposed in this pull request?

`interfaces.scala` was getting big. This just moves the biggest class in 
there to a new file for cleanliness.

## How was this patch tested?

Just moving things around.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/andrewor14/spark move-external-catalog

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/12721.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #12721


commit 42c858d4e1b8ad5c49d3c65b218d38f5c0c3ebab
Author: Andrew Or 
Date:   2016-04-26T23:39:09Z

Just move 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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org