[GitHub] spark issue #15303: [SPARK-17671] Changed implementation of HistoryServer.ge...

2016-10-02 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/15303 It's cleaner, but I have no strong opinions --- 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

[GitHub] spark issue #15303: [SPARK-17671] Changed implementation of HistoryServer.ge...

2016-09-30 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15303 See my comments on the previous PR. I think this is much more simply handled by returning an Iterator over apps from the beginning, calling application.values.iterator instead of application.values

[GitHub] spark issue #15303: [SPARK-17671] Changed implementation of HistoryServer.ge...

2016-09-29 Thread wgtmac
Github user wgtmac commented on the issue: https://github.com/apache/spark/pull/15303 @srowen @ajbozarth I created this PR without adding any new API. Just rewrote the way getApplicationList constructing the iterator. Can you guys take a look? Thanks! --- If your project is set up