Skip to content
This repository has been archived by the owner on Sep 20, 2022. It is now read-only.

[HIVEMALL-37][SPARK] Support a SST-based change-point detector in DataFrame/Spark #30

Closed
wants to merge 1 commit into from

Conversation

maropu
Copy link
Member

@maropu maropu commented Jan 26, 2017

What changes were proposed in this pull request?

This pr supported a SST-based change-point detector as sst in HivemallOps.

What type of PR is it?

Feature

What is the Jira issue?

https://issues.apache.org/jira/browse/HIVEMALL-37

How was this patch tested?

Added a test in HivemallOpsSuite.

@coveralls
Copy link

coveralls commented Jan 26, 2017

Coverage Status

Coverage remained the same at 35.842% when pulling 20b36f6 on maropu:HIVEMALL-37 into 1801a62 on apache:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 35.842% when pulling 20b36f6 on maropu:HIVEMALL-37 into 1801a62 on apache:master.

@myui
Copy link
Member

myui commented Jan 26, 2017

LGTM. You can merge this PR.

@myui
Copy link
Member

myui commented Jan 26, 2017

@maropu Could you add [SPARK] after [HIVEMALL-37] in the title?

@maropu maropu changed the title [HIVEMALL-37] Support a SST-based change-point detector in DataFrame/Spark [HIVEMALL-37][SPARK] Support a SST-based change-point detector in DataFrame/Spark Jan 26, 2017
@maropu
Copy link
Member Author

maropu commented Jan 27, 2017

I'll merge this after #29 merged because they have conflicts between each other.

@myui
Copy link
Member

myui commented Jan 27, 2017

@maropu LGTM. Please merge this PR and close the JIRA ticket.

@maropu
Copy link
Member Author

maropu commented Jan 27, 2017

Merged

@asfgit asfgit closed this in 892ae1c Jan 27, 2017
@coveralls
Copy link

coveralls commented Jan 27, 2017

Coverage Status

Coverage remained the same at 35.842% when pulling 34489c1 on maropu:HIVEMALL-37 into 88fae52 on apache:master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants