[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/1620 --- 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 enab

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-28 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/1620#issuecomment-50419644 Oh, really "lucky" these days :) --- 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 no

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-28 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/1620#issuecomment-50419434 https://git-wip-us.apache.org/repos/asf?p=spark.git;a=commit;h=a7a9d14479ea6421513a962ff0f45cb969368bab GitHub is lagging... We filled a bug with infra. On J

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-28 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/1620#issuecomment-50419333 Hmm, didn't find related code in master, and this PR is still open, seems it's not successfully merged yet? --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-28 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/1620#issuecomment-50383782 Hopefully third times a charm :) Merged! --- 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] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1620#issuecomment-50382841 QA results for PR 1620:- This patch PASSES unit tests.- This patch merges cleanly- This patch adds the following public classes (experimental):class SparkSQLOperationMana

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1620#issuecomment-50372024 QA tests have started for PR 1620. This patch merges cleanly. View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17287/consoleFull --- If

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-28 Thread liancheng
GitHub user liancheng opened a pull request: https://github.com/apache/spark/pull/1620 [SPARK-2410][SQL] Merging Hive Thrift/JDBC server (with Maven profile fix) JIRA issue: [SPARK-2410](https://issues.apache.org/jira/browse/SPARK-2410) Another try for #1399 & #1600. Those t

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-27 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50293934 Ouch... This is really embarrassing... OK, I'll try to refactor those possibly flaky test code inherited from Shark (e.g. hard coded timeout, etc.). --- If your projec

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-27 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50292705 What about this one: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17259/consoleFull --- If your project is set up for it, you can reply to this em

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-27 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50292629 Thanks @pwendell, just checked all failed Jenkins builds ever since then, fortunately none was caused by this PR (phew...). --- If your project is set up for it, you c

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/1600 --- 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 enab

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-27 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50284058 Okay - lets try this again. I can merge 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 proje

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50225967 QA results for PR 1600:- This patch PASSES unit tests.- This patch merges cleanly- This patch adds the following public classes (experimental):class SparkSQLOperationMana

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50224608 QA tests have started for PR 1600. This patch merges cleanly. View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17217/consoleFull --- If

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread concretevitamin
Github user concretevitamin commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50224519 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

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50224484 @marmbrus The build failure was caused by PySpark, please help re-test this, thanks! --- If your project is set up for it, you can reply to this email and have your re

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50223630 QA results for PR 1600:- This patch FAILED unit tests.- This patch merges cleanly- This patch adds the following public classes (experimental):class SparkSQLOperationMana

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/1399#issuecomment-50222515 Opened #1600 to replace this PR. Hope Mr. Jenkins accepts it gently... --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1600#issuecomment-50222427 QA tests have started for PR 1600. This patch merges cleanly. View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17215/consoleFull --- If

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread liancheng
GitHub user liancheng opened a pull request: https://github.com/apache/spark/pull/1600 [SPARK-2410][SQL] Merging Hive Thrift/JDBC server (This is a replacement of #1399, trying to fix potential `HiveThriftServer2` port collision between parallel builds. Please refer to [these comm

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/1399#issuecomment-50219827 Ah, thanks, then I think this might be the problem. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If y

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/1399#issuecomment-50219743 Is there a fixed port or something that we use? On Jenkins we run multiple tests at the same time so if you have a single fixed port, tests can fail. --- If your pro

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/1399#issuecomment-50219674 @marmbrus Sorry for the trouble. It's weird that the build suddenly breaks continuously after several heathy builds. Can't reproduce the failure locally right now, I'll

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/1399#issuecomment-50212572 Hey this was making Jenkins fail so I reverted it. We should investigate and try again. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/1399 --- 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 enab

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/1399#issuecomment-50193232 Awesome, thanks for all of your work on this! I've merged this into master. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1399#issuecomment-50192198 QA results for PR 1399:- This patch PASSES unit tests.- This patch merges cleanly- This patch adds the following public classes (experimental):class SparkSQLOperationMana

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/1399#issuecomment-50182685 Hey @pwendell, changes related to [SPARK-2678](https://issues.apache.org/jira/browse/SPARK-2678) are reverted as we discussed offline. The beeline script is now calling

[GitHub] spark pull request: [SPARK-2410][SQL] Merging Hive Thrift/JDBC ser...

2014-07-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1399#issuecomment-50182595 QA tests have started for PR 1399. This patch merges cleanly. View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17188/consoleFull --- If