[GitHub] spark pull request: [SPARK-10113][SQL] Support for unsigned Parque...
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/9646#issuecomment-155992080 Would you please rename the PR title to "Explicit error message for unsigned Parquet logical types"? Since this PR doesn't introduce "support" for unsigned types. Otherwise LGTM. Thanks! --- 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-10113][SQL] Support for unsigned Parque...
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9646#issuecomment-155986820 cc @liancheng --- 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-10113][SQL] Support for unsigned Parque...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9646#issuecomment-155986526 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45693/ 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-10113][SQL] Support for unsigned Parque...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9646#issuecomment-155986525 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-10113][SQL] Support for unsigned Parque...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9646#issuecomment-155986437 **[Test build #45693 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45693/consoleFull)** for PR 9646 at commit [`2752c14`](https://github.com/apache/spark/commit/2752c14331539bda7ccd7e1bc5a2887da2a2b4cd). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- 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-10113][SQL] Support for unsigned Parque...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9646#issuecomment-155966525 **[Test build #45693 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45693/consoleFull)** for PR 9646 at commit [`2752c14`](https://github.com/apache/spark/commit/2752c14331539bda7ccd7e1bc5a2887da2a2b4cd). --- 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-10113][SQL] Support for unsigned Parque...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9646#issuecomment-155964973 Merged build started. --- 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-10113][SQL] Support for unsigned Parque...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9646#issuecomment-155964957 Merged build triggered. --- 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-10113][SQL] Support for unsigned Parque...
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/9646 [SPARK-10113][SQL] Support for unsigned Parquet logical types Parquet supports some unsigned datatypes. However, Since Spark does not support unsigned datatypes, it needs to emit an exception with a clear message rather then with the one saying illegal datatype. You can merge this pull request into a Git repository by running: $ git pull https://github.com/HyukjinKwon/spark SPARK-10113 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/9646.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 #9646 commit 49e162480e33a0379e1b04979666533e66e33f6e Author: hyukjinkwon Date: 2015-11-12T01:00:56Z [SPARK-10113][SQL] Support for unsigned Parquet logical types commit 2752c14331539bda7ccd7e1bc5a2887da2a2b4cd Author: hyukjinkwon Date: 2015-11-12T01:01:26Z [SPARK-10113][SQL] Add a variable for readability --- 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