[GitHub] spark issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user JoshRosen commented on the issue: https://github.com/apache/spark/pull/15169 LGTM. I walked through the code in IntelliJ and have verified that the added unit test fails without this change. It's weird that `s3a` would return a non-absolute path when producing the qualified path for the bucket root, but this fix seems correct to me. --- 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15169 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15169 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65731/ 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15169 **[Test build #65731 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65731/consoleFull)** for PR 15169 at commit [`ef781ad`](https://github.com/apache/spark/commit/ef781ad67468b641f9e2ded12a4394e0c05f082c). * 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15169 **[Test build #65731 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65731/consoleFull)** for PR 15169 at commit [`ef781ad`](https://github.com/apache/spark/commit/ef781ad67468b641f9e2ded12a4394e0c05f082c). --- 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15169 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65689/ 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15169 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15169 **[Test build #65689 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65689/consoleFull)** for PR 15169 at commit [`f5a9c12`](https://github.com/apache/spark/commit/f5a9c12536a16e373c33b8e84bcd07126b77ccaf). * 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15169 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65683/ 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15169 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15169 **[Test build #65683 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65683/consoleFull)** for PR 15169 at commit [`e168a63`](https://github.com/apache/spark/commit/e168a6311cf85b00c187cc64eca605ea9e5a8a6d). * 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15169 **[Test build #65689 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65689/consoleFull)** for PR 15169 at commit [`f5a9c12`](https://github.com/apache/spark/commit/f5a9c12536a16e373c33b8e84bcd07126b77ccaf). --- 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15169 **[Test build #65688 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65688/consoleFull)** for PR 15169 at commit [`97a85d9`](https://github.com/apache/spark/commit/97a85d9970fda23ced2e5fcbab8b8906678ae50e). * This patch **fails Scala style 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15169 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65688/ 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15169 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15169 **[Test build #65688 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65688/consoleFull)** for PR 15169 at commit [`97a85d9`](https://github.com/apache/spark/commit/97a85d9970fda23ced2e5fcbab8b8906678ae50e). --- 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15169 **[Test build #65683 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65683/consoleFull)** for PR 15169 at commit [`e168a63`](https://github.com/apache/spark/commit/e168a6311cf85b00c187cc64eca605ea9e5a8a6d). --- 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 issue #15169: [SPARK-17613] S3A base paths with no '/' at the end retu...
Github user brkyvz commented on the issue: https://github.com/apache/spark/pull/15169 cc @tdas This is the fix for the issue you were facing. --- 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
