zhuqi-lucas commented on PR #16196:
URL: https://github.com/apache/datafusion/pull/16196#issuecomment-2952375579

   > @zhuqi-lucas, I wanted to make a few final finishing touches as we gave a 
chance in case @alamb wants to take a final look. I changed the config 
terminology from "frequency" to "period" because the former was kind of a 
misnomer. I also did some refactoring to remove some code repeatitions. Can you 
please double check to make sure all looks good on your end? Thanks.
   
   Thank you @ozankabak , the final refractor and name change looks good to me. 
Yeah , let's wait @alamb to get the final look. Thanks! 


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

To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to