Github user tillrohrmann commented on the pull request:
https://github.com/apache/flink/pull/861#issuecomment-141898541
Sorry Sachin, my backlog is currently really long. But I also would like to
take another look because you completely rewrote the whole implementation.
Please bear a little bit longer with us. I will really try to review your
PR so that your work was not in vain.
On Fri, Sep 18, 2015 at 4:39 PM, Sachin Goel <notificati...@github.com>
wrote:
> Rebased to reflect the changes in the scala utility functions.
> Travis failure on an unrelated error. Reported at jira id 2700.
>
> This has already undergone three reviews, and been open for almost four
> months (as part of #710 <https://github.com/apache/flink/pull/710> too).
> If this functionality is not required, I can close the PR; that'd however
> automatically mean closing #710 <https://github.com/apache/flink/pull/710>
> too since this is essential to implementing a Decision Tree. I'd prefer
not
> to though as I've spent a lot of time optimizing this.
>
> @chiwanpark <https://github.com/chiwanpark> @tillrohrmann
> <https://github.com/tillrohrmann>
>
> â
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/flink/pull/861#issuecomment-141471679>.
>
---
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.
---
