hit-lacus commented on pull request #1354:
URL: https://github.com/apache/kylin/pull/1354#issuecomment-672890438
In general, this look good to me.
This is an automated message from the Apache Git Service.
To respond to the
hit-lacus commented on pull request #1354:
URL: https://github.com/apache/kylin/pull/1354#issuecomment-672888369
This patch has some unpredictable performance lose impact on both build
engine and query engine. But I think we can merge it and do a benchmark to
analyse the difference .