[GitHub] spark issue #15686: [MINOR][DOC] Remove spaces following slashs
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15686 Thank you for merging, @srowen . --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15686 Merged 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 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15686 Never mind, @HyunjinKwon . I was also curious about AppVoyer failure. :) --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15686 @dongjoon-hyun I am sorry for unrelated comments here. All these comments are not related with this PR. @shivaram Sure, Let me try to create a JIRA. I will cc you. We might be able to talk more there. --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/15686 Yeah I think it'll be a good idea to know the error rate we hit. @HyukjinKwon It might also be good to create an INFRA ticket to see if we can re-trigger the AppVeyor builds something --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15686 There are three problems with it. - it starts the build when someone merges the latest upstream (not rebases) when pushing more commits in its PR (as the merged one usually has the changes in R) - this case seems fine though because usually it suceeds to build. Also actually I asked this case to AppVeyor but I haven't received the reply. - it sometimes failes to build (it is relatively rare I believe). - it leaves a failure mark on other branches when someone opens a PR from a branch to another branch. I am neutral because it seems the cases are not often. I may open a JIRA if I incline toward disabling this or please feel free to create a JIRA if anyone feels so. (Let me please cc @shivaram. I would like him to know about this). --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15686 This change seems fine. If Appveyor is encountering errors regularly, we might consider disabling it, just because it wouldn't be giving useful information if most failures were spurious. --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15686 I believe the message indicates the same case with the PR 15673 but it seems spurious in this case. --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15686 @rxin, it seems spurious. The message seems meaning the failure when the commit is virtually not mergeable[1]. It seems it fails time to time for various reasons. For example, in some cases, it seems failing to clone due to network problem within AppVeyor. it is able to re-trigger via Web UI but it seems committers are also not allowed to access to it. Due to this problem, I temporarily made a bunch of scripts[3] to manually launch another build via @spark-test account. I can run the build and leave the comment above until we find a better way to handle it. (treat me like a bot) [1]http://help.appveyor.com/discussions/problems/4912-merge-commit-does-not-start-build-in-the-repo-i-make-pull-request-for [2]https://ci.appveyor.com/project/ApacheSoftwareFoundation/spark/build/256-master [3]https://github.com/HyukjinKwon/spark-appveyor --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15686 @HyukjinKwon why are the appveyor checks failing? --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15686 Build started: [SparkR] `ALL` [![PR-15686](https://ci.appveyor.com/api/projects/status/github/spark-test/spark?branch=1D4EC6E8-F2CF-4585-9745-0AE5956F211C=true)](https://ci.appveyor.com/project/spark-test/spark/branch/1D4EC6E8-F2CF-4585-9745-0AE5956F211C) Diff: https://github.com/apache/spark/compare/master...spark-test:1D4EC6E8-F2CF-4585-9745-0AE5956F211C :) --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15686 Thank you for review and approval, @srowen ! --- 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15686 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67782/ 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15686 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15686 **[Test build #67782 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67782/consoleFull)** for PR 15686 at commit [`b385a4c`](https://github.com/apache/spark/commit/b385a4c4e1d20550285d2269c14c738322555f50). * 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 #15686: [MINOR][DOC] Remove spaces following slashs
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15686 **[Test build #67782 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67782/consoleFull)** for PR 15686 at commit [`b385a4c`](https://github.com/apache/spark/commit/b385a4c4e1d20550285d2269c14c738322555f50). --- 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