[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21981 @hhbyyh BTW, there's typo for the JIRA ticket in the title. Not a big deal though better to match. --- - To unsubscribe,

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21981 Merged to master. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21981 I am not sure. ML is not my area but I am pretty sure who you know are basically who I know .. ping me if that's minor or trivial like this. I can review and merge. ---

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21981 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21981 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94105/ Test PASSed. ---

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21981 **[Test build #94105 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94105/testReport)** for PR 21981 at commit

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user hhbyyh commented on the issue: https://github.com/apache/spark/pull/21981 BTW, @HyukjinKwon, do you know who's still reviewing the ML PRs? I have a few old PRs and I really want to know which're considered meaningful. ---

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user hhbyyh commented on the issue: https://github.com/apache/spark/pull/21981 Thanks for the review @HyukjinKwon. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21981 **[Test build #94105 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94105/testReport)** for PR 21981 at commit

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21981 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21981 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user hhbyyh commented on the issue: https://github.com/apache/spark/pull/21981 Ah, this triggers the doc check. Updating. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21981 Looks we should fix the doc in PrefixSpan too. LGTM if the check pass. --- - To unsubscribe, e-mail:

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21981 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21981 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21981 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21981 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94102/ Test FAILed. ---

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21981 **[Test build #94102 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94102/testReport)** for PR 21981 at commit

[GitHub] spark issue #21981: [SAPRK-25011][ML]add prefix to __all__ in fpm.py

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21981 **[Test build #94102 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94102/testReport)** for PR 21981 at commit