[GitHub] flink issue #2628: [FLINK-3722] [runtime] Don't / and % when sorting

2016-10-20 Thread ggevay
Github user ggevay commented on the issue: https://github.com/apache/flink/pull/2628 > I transcribed Quicksort so as to remove considerations of Java performance and inlining. It was not clear to me that if we encapsulated the index, page number, and page offset into an object that

[GitHub] flink issue #2628: [FLINK-3722] [runtime] Don't / and % when sorting

2016-10-20 Thread greghogan
Github user greghogan commented on the issue: https://github.com/apache/flink/pull/2628 Thanks @ggevay for reviewing. I added a commit with additional comments. I transcribed `Quicksort` so as to remove considerations of Java performance and inlining. It was not clear to me

[GitHub] flink issue #2628: [FLINK-3722] [runtime] Don't / and % when sorting

2016-10-14 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/2628 Thanks for these cool experiments. Looks like a nice improvement. I'll try to dig into your code in the next days... --- If your project is set up for it, you can reply to this email and