liyafan82 commented on issue #8397: [FLINK-11421][Table SQL/Runtime]Add 
compilation options to allow comp…
URL: https://github.com/apache/flink/pull/8397#issuecomment-493927451
 
 
   @JingsongLi Thank you so much for taking time to verify our results.
   For 1). I think it is generally expected. JCA takes 10x longer for 
compilation. We have some techniques to hide this time, but they are not 
implemented in this PR.
   For 2). I think at least, your results reveal that the improvements are 
unstable. Even if you can reproduce the 2% - 7% performance improvements, such 
improvements are not significant enough to help improve the performance of 
row-store. So I think more investigations are required for us. 
   
   @KurtYoung the article is impressive. Thank you so much for your suggestion.
   
   Anyway, I would like to thank you so much for all your helpful suggestions 
and valuable comments. Let's work together to make row-store better, to make 
Flink better. 

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

Reply via email to