Re: Improve performance of call system.currentTimeMillis()

2015-08-12 Thread Ufuk Celebi
On Wed, Aug 12, 2015 at 3:57 PM, Stephan Ewen wrote: > I curious what the results are! Same here! :-)

Re: Improve performance of call system.currentTimeMillis()

2015-08-12 Thread Stephan Ewen
@ffbin: Would this be a JVM singleton that updates a static field every millisecond? It is hard to say at this point whether this eliminates a bottleneck, but I think it is fine to try. To evaluate this, you could use a streaming job that attaches a timestamp to every record, and measure the diff

Re: Improve performance of call system.currentTimeMillis()

2015-08-12 Thread Maximilian Michels
I second Ufuk and Chensnay. Please provide us with a benchmark. I have a hard time to believe your implementation, along with the overhead that comes with it, will improve the streaming performance. Please, feel free to prove us wrong :) On Wed, Aug 12, 2015 at 11:48 AM, Chesnay Schepler < chesna

Re: Improve performance of call system.currentTimeMillis()

2015-08-12 Thread Chesnay Schepler
-.- if you look at into this issue that you opened https://issues.apache.org/jira/browse/FLINK-2471 you were given opinions from 2 separate people, with arguments, that the performance improvement(TBD) is either a) nonexistant or b) negligible. All you did was disregard those basically saying

Re: Improve performance of call system.currentTimeMillis()

2015-08-12 Thread Ufuk Celebi
Hey Fengbin, did you run a program and noticed some kind of performance degradation because of this? If yes, could you provide some details? If not, I would suggest to not do this. I can see how this "improves" performance in micro benchmarks, but not how it would affect the overall performance o