You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by sethah <gi...@git.apache.org> on 2017/05/17 23:21:50 UTC

[GitHub] spark issue #17094: [SPARK-19762][ML] Hierarchy for consolidating ML aggrega...

Github user sethah commented on the issue:

    https://github.com/apache/spark/pull/17094
  
    ping! @MLnick @jkbradley @yanboliang @hhbyyh 
    
    Is there any interest in this? I actually think this cleanup will be a precursor to several different improvements (adding more optimized aggregators, adding optimization library) and that it will be very useful. IMO it's an important change. Otherwise, we keep slapping layers on the current implementation and the code length and complexity keeps growing.
    
    I'm happy to take suggestions, make changes, or discuss it further. Thoughts?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org