[GitHub] spark pull request: [SPARK-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126847980 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126847975 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126849534 [Test build #39308 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39308/console) for PR 4716 at commit [`c572417`](https://github.com/apache/spark/commit/c572417ea232527bad046885c7f61a7e97bc66bb). * This patch **fails Spark unit 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126849547 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126840384 [Test build #39300 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39300/console) for PR 4716 at commit [`76f40ff`](https://github.com/apache/spark/commit/76f40ff1e84bfdd620d9e40232d5a321c06271fb). * This patch **fails Spark unit 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126840397 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126848164 [Test build #39308 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39308/consoleFull) for PR 4716 at commit [`c572417`](https://github.com/apache/spark/commit/c572417ea232527bad046885c7f61a7e97bc66bb). --- 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126832376 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126832678 [Test build #39295 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39295/consoleFull) for PR 4716 at commit [`93a0d34`](https://github.com/apache/spark/commit/93a0d342082850454f9c181062312ca4084752c2). --- 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-3147][MLLib] A/B testing
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126832384 @mengxr Just rebased this onto 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126832362 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126833294 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126833287 [Test build #39295 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39295/console) for PR 4716 at commit [`93a0d34`](https://github.com/apache/spark/commit/93a0d342082850454f9c181062312ca4084752c2). * 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 pull request: [SPARK-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126837892 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126837902 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-126838273 [Test build #39300 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39300/consoleFull) for PR 4716 at commit [`76f40ff`](https://github.com/apache/spark/commit/76f40ff1e84bfdd620d9e40232d5a321c06271fb). --- 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-100987876 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-100987868 [Test build #32412 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32412/consoleFull) for PR 4716 at commit [`80c2211`](https://github.com/apache/spark/commit/80c22112613a0494f8f0e1ee6806fdccd6fb89ec). * 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 pull request: [SPARK-3147][MLLib] A/B testing
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101054028 @feynmanliang can you close this PR? --- 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-3147][MLLib] A/B testing
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101055170 @feynmanliang GitHub messed up with the diff. Could you merge the current master and push an update? Another way to refresh the diff is to close this PR first and then re-open it, as @rxin suggested. 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101080085 [Test build #32433 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32433/consoleFull) for PR 4716 at commit [`f4de414`](https://github.com/apache/spark/commit/f4de414d61386b9f5597eed52d9a1c3b16244c25). * 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101080096 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32433/ 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101080094 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101066848 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101066874 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101066934 [Test build #32432 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32432/consoleFull) for PR 4716 at commit [`a36ba79`](https://github.com/apache/spark/commit/a36ba79250073e0060d36f6af74b70675271a1dc). --- 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101067159 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32432/ 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101067154 [Test build #32432 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32432/consoleFull) for PR 4716 at commit [`a36ba79`](https://github.com/apache/spark/commit/a36ba79250073e0060d36f6af74b70675271a1dc). * 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 pull request: [SPARK-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101067157 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101067529 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-3147][MLLib] A/B testing
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101067479 @mengxr Fixed the diff, also pushed a commit to address the style guide issues. --- 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-101067503 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-10106 [Test build #32433 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32433/consoleFull) for PR 4716 at commit [`f4de414`](https://github.com/apache/spark/commit/f4de414d61386b9f5597eed52d9a1c3b16244c25). --- 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-3147][MLLib] A/B testing
Github user feynmanliang closed the pull request at: https://github.com/apache/spark/pull/4716 --- 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-3147][MLLib] A/B testing
GitHub user feynmanliang reopened a pull request: https://github.com/apache/spark/pull/4716 [SPARK-3147][MLLib] A/B testing Implementation of A/B testing using Streaming API. This contribution is my original work and I license the work to the project under the project's open source license. You can merge this pull request into a Git repository by running: $ git pull https://github.com/feynmanliang/spark ab_testing Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/4716.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 #4716 commit a6cdc201dfed5342e7146613509e2a2eb8551a73 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-10T19:36:27Z Add broken implementation of AB testing. commit b730864885229c9aa5250c85708c13e2022f0df4 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-10T21:07:29Z Fix AB testing implementation and add unit tests. commit 24713bbcda298e603e4937eef489bbfc42a2d4e8 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-10T22:54:26Z Extract t-testing code out of OnlineABTesting. commit 4d1808c5e7dae4460fe6e8bed9da27174babdc0a Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T04:56:30Z Add peace period for dropping first k entries of each A/B group. commit e42cf9a48ad2bd1e66c3a31b175c5597ae24bbb8 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T05:09:18Z Add numDim to MultivariateOnlineSummarizer. commit 440badd8045ed7bb7f65d9195bc5364c7de5464f Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T06:02:41Z Refactored ABTestingMethod into sealed trait. commit 728cb8c78da6144a0ac6f1524ddd020d1860fe8d Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T06:29:49Z Add (non-sliding) testing window functionality. commit a8937711c6b60e7af523373b798efe60393789a8 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T15:19:46Z Fix peace period implementation. commit 61aefd4e28896f2e0e199870e007687bbeee5f3a Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T17:31:05Z Fix test window batching. commit 2f140e98325b3c6dbbfa922a12d44d22d72ba304 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T17:56:15Z Handle (inelegantly) closure capture for ABTestMethod commit 7f2eb22c7cdbabaf0d04aee25752a3cf01256727 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T18:20:19Z Improve handling of OnlineABTestMethod closure by moving DStream processing method into Serializable class. commit 50ccffd2dd8afbf34f0065aba92702bf27d74fa4 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T18:52:37Z Fixed flaky peacePeriod test. commit c7c2fb455fbfbb8d4ccbb40a42a018e8283fa9fb Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T20:39:58Z Add ScalaDocs and format to style guide. commit d4852a8c02e397ac6c126844501444dcb51faaca Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-12T00:02:20Z Add OnlineABTestExample. commit a3e391db3a8dfda477cf4df6e929e490a05a3d6a Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-12T00:03:12Z Format code to style guide. commit 89ea01614145814ab8f333e327d88e9530b69865 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-19T04:43:36Z Switch MultivariateOnlineSummarizer to univariate StatsCounter. commit 64b13c468462230bb92593191c4ebf39d7e588f0 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-19T14:45:15Z Reduce number of passes in pairSummaries. commit 0cc280d50ef6dbf811d82bc32530f87dfea89ff7 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-19T14:45:41Z Add test for behavior when missing data from one group. commit a36ba79250073e0060d36f6af74b70675271a1dc Author: Feynman Liang feynman.li...@gmail.com Date: 2015-02-24T03:56:54Z Remove numDim from MultivariateOnlineSummarizer. --- 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-100987085 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-100987073 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-100987880 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32412/ 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-100987234 [Test build #32412 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32412/consoleFull) for PR 4716 at commit [`80c2211`](https://github.com/apache/spark/commit/80c22112613a0494f8f0e1ee6806fdccd6fb89ec). --- 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-3147][MLLib] A/B testing
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-90734623 @freeman-lab Do you want to make a pass on this PR? --- 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-3147][MLLib] A/B testing
Github user freeman-lab commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-90807615 @mengxr @feynmanliang sure thing! This looks really cool, will try to go through it in the next couple 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 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-3147][MLLib] A/B testing
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75543022 `[error] * abstract method numDim()Int in interface org.apache.spark.mllib.stat.MultivariateStatisticalSummary does not have a correspondent in old version` Would it be bett --- 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-3147][MLLib] A/B testing
Github user feynmanliang closed the pull request at: https://github.com/apache/spark/pull/4716 --- 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-3147][MLLib] A/B testing
GitHub user feynmanliang reopened a pull request: https://github.com/apache/spark/pull/4716 [SPARK-3147][MLLib] A/B testing Implementation of A/B testing using Streaming API. This contribution is my original work and I license the work to the project under the project's open source license. You can merge this pull request into a Git repository by running: $ git pull https://github.com/feynmanliang/spark ab_testing Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/4716.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 #4716 commit 105401a89216516565236f59a66a22cc91830686 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-10T19:36:27Z Add broken implementation of AB testing. commit cb73e790c435a4819fb62bc6c37717f4b882aee4 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-10T21:07:29Z Fix AB testing implementation and add unit tests. commit e0d5beccf54914ebdc5663dbe4ba71944f3183e2 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-10T22:54:26Z Extract t-testing code out of OnlineABTesting. commit 2100de641a2e86efeaa0f559500c7ced6f7d51a9 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T04:56:30Z Add peace period for dropping first k entries of each A/B group. commit 708380e980ed46ac1beb7665f7854fcf36ebc403 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T05:09:18Z Add numDim to MultivariateOnlineSummarizer. commit ec7f700fbca15d84bba126edaaa50d53ce5fc7be Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T06:02:41Z Refactored ABTestingMethod into sealed trait. commit 3f19e15aa3b7056262b601686643ed962846cdc3 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T06:29:49Z Add (non-sliding) testing window functionality. commit c56f9237aa81a70e8572e2ecb851ebaf5cdfa473 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T15:19:46Z Fix peace period implementation. commit 0d738815eb1cd49096112d8be7e9124345af0604 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T17:31:05Z Fix test window batching. commit abf59d5e8f817f847af77aef7514fb740dbbf69d Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T17:56:15Z Handle (inelegantly) closure capture for ABTestMethod commit e05eaaf3bb21bbed4c123d9ec6514e84ae75adcb Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T18:20:19Z Improve handling of OnlineABTestMethod closure by moving DStream processing method into Serializable class. commit 964a555746273a3afa542e34fdc6b86be60a5db9 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T18:52:37Z Fixed flaky peacePeriod test. commit 79c1d44c6232b0a4af5df4dc14cdc83919cfdea9 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-11T20:39:58Z Add ScalaDocs and format to style guide. commit e030c12337dce99abcf26f7d02c5d00a78f58c9b Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-12T00:02:20Z Add OnlineABTestExample. commit e8e1f82b16fbdd8446e21b32bb39b413e1ae30d1 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-12T00:03:12Z Format code to style guide. commit 17eef4eb22d918198dd03f2a931f009863fadcf5 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-19T04:43:36Z Switch MultivariateOnlineSummarizer to univariate StatsCounter. commit a2ad38be8a77eef045581282b3dbc9d6a1544870 Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-19T14:45:15Z Reduce number of passes in pairSummaries. commit 4bb8636e5317a542ff0b29270548bd933199c6eb Author: Feynman Liang feynman.li...@gmail.com Date: 2015-01-19T14:45:41Z Add test for behavior when missing data from one group. --- 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-3147][MLLib] A/B testing
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75580259 Let's remove `numDim`. --- 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75503155 [Test build #27851 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27851/consoleFull) for PR 4716 at commit [`4bb8636`](https://github.com/apache/spark/commit/4bb8636e5317a542ff0b29270548bd933199c6eb). * This patch merges cleanly. --- 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75511258 [Test build #27851 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27851/consoleFull) for PR 4716 at commit [`4bb8636`](https://github.com/apache/spark/commit/4bb8636e5317a542ff0b29270548bd933199c6eb). * This patch **fails MiMa 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75511267 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27851/ 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-3147][MLLib] A/B testing
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75502732 ok to test --- 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-3147][MLLib] A/B testing
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75502716 add to whitelist --- 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75701749 [Test build #27880 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27880/consoleFull) for PR 4716 at commit [`7ce63c8`](https://github.com/apache/spark/commit/7ce63c813bb2b82b4771ffbc6d7ec5bb5900ce94). * 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-3147][MLLib] A/B testing
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75701760 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27880/ 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-3147][MLLib] A/B testing
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4716#issuecomment-75695983 [Test build #27880 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27880/consoleFull) for PR 4716 at commit [`7ce63c8`](https://github.com/apache/spark/commit/7ce63c813bb2b82b4771ffbc6d7ec5bb5900ce94). * This patch merges cleanly. --- 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