[GitHub] [spark] srowen commented on issue #25178: [SPARK-28421][ML] SparseVector.apply performance optimization

2019-07-25 Thread GitBox
srowen commented on issue #25178: [SPARK-28421][ML] SparseVector.apply performance optimization URL: https://github.com/apache/spark/pull/25178#issuecomment-515033303 OK, also manually merged to 2.4 in https://github.com/apache/spark/commit/a285c0d4fe071ceed05d8ea693febfbac5962d64

[GitHub] [spark] srowen commented on issue #25178: [SPARK-28421][ML] SparseVector.apply performance optimization

2019-07-24 Thread GitBox
srowen commented on issue #25178: [SPARK-28421][ML] SparseVector.apply performance optimization URL: https://github.com/apache/spark/pull/25178#issuecomment-514605032 Hm, I can't seem to back-port a merged PR with the merge script right now. I've seen this before. @dongjoon-hyun are you

[GitHub] [spark] srowen commented on issue #25178: [SPARK-28421][ML] SparseVector.apply performance optimization

2019-07-23 Thread GitBox
srowen commented on issue #25178: [SPARK-28421][ML] SparseVector.apply performance optimization URL: https://github.com/apache/spark/pull/25178#issuecomment-514465540 I'm OK with it. It should be a pretty safe optimization.

[GitHub] [spark] srowen commented on issue #25178: [SPARK-28421][ML] SparseVector.apply performance optimization

2019-07-23 Thread GitBox
srowen commented on issue #25178: [SPARK-28421][ML] SparseVector.apply performance optimization URL: https://github.com/apache/spark/pull/25178#issuecomment-514441092 Merged to master This is an automated message from the