hit-lacus commented on pull request #1573: URL: https://github.com/apache/kylin/pull/1573#issuecomment-781912579
Firstly, I feel good about this patch. Secondly, last time you said you will verify if **lambda mode** (use data from hive to refresh/overwrite data from Kafka) works as you expected. Since the fact that these two feature are related (on timezone), I worried about that if you have an another patch on **lambda mode**, I have to test timezone related test case twice. ---------------------------------------------------------------- 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: us...@infra.apache.org