Kyligence / calcite

a tailored Apache Calcite for Apache Kylin, more details at http://mail-archives.apache.org/mod_mbox/kylin-dev/201704.mbox/%3CCAF7etT=wEBPKm4C_6ffssQ0=kEhD=j1jz3O9DpjC+Zu9xWU=5A@mail.gmail.com%3E . Isn't AtopCalcite in Kylin enough? It depends on tomcat feature that's no longer supported in tomcat 8
Apache License 2.0
14 stars 53 forks source link

Calcite 升级1.16之后,SQL语法树转化成DAG对KAP的影响 #8

Open vaful opened 6 years ago

vaful commented 6 years ago

主要是会影响到Context的划分,Kap IT下面的这些Query会出现问题:

  1. org.apache.kylin.query.ITKylinQueryTest#testGroupingQuery
  2. io.kyligence.kap.ITKapKylinQueryTest#testMagine(sql-04)
  3. io.kyligence.kap.ITKapKylinQueryTest#testMultiModelQuery(sql-01)
  4. org.apache.kylin.query.ITKylinQueryTest#testSubQuery(query03)
  5. org.apache.kylin.query.ITKylinQueryTest#testValues(query01)
vaful commented 6 years ago

解决办法: 将这个树生成DAG的commit revert掉,https://github.com/Kyligence/calcite/commit/c0f912ddff717eaedb9a2d652e95f232d10e207f

[CALCITE-2116] In HepPlanner, ensure that common sub-expressions have the same digest (LeoWangLZ)" This reverts commit c0f912ddff717eaedb9a2d652e95f232d10e207f.