neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax URL: https://github.com/apache/jena/pull/568#issuecomment-496912773 so I did a few more tests and I have to say we might actually be better off to roll our own here for the median function in ARQ and remove Apache Common Math dependencies for now. I get much faster results with a plain vanilla implementation of median with Arrays.sort over the Apache commons Median.evaluate approach. I might fail to see the benefits of using Apache Commons Math here but I used to be turned off by the performance of Arrays.sort in the past. That seems to have changed since JDK7 and further performance improvements with JDK 12 now. Again I will try to further evaluate this with the Apache Commons Maths project.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
