gianm opened a new pull request #10904:
URL: https://github.com/apache/druid/pull/10904


   Saves creation of unnecessary DateTime objects in timestamp_floor and
   timestamp_ceil expressions.
   
   I was running a query that had a `group by floor(__time to year)` in it and 
was greeted by the following flame graph: 
   
   
![image](https://user-images.githubusercontent.com/1214075/108477473-1de9b780-7248-11eb-9eb9-6fb24bec6381.png)
   
   33% of the time is spent on `DateTime.<init>`. It's not necessary, because 
the PeriodGranularity code is doing the hard work on the primitive milliseconds.


----------------------------------------------------------------
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]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to