You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@kyuubi.apache.org by GitBox <gi...@apache.org> on 2022/07/22 11:41:53 UTC

[GitHub] [incubator-kyuubi] pan3793 commented on pull request #3106: Correct `RelMetadataProvider` used in flink-sql-engine

pan3793 commented on PR #3106:
URL: https://github.com/apache/incubator-kyuubi/pull/3106#issuecomment-1192485779

   > I found that some unit test case failed with flink version 1.15 due to the difference between flink 1.14 and 1.15.
   
   This is because Kyuubi engine uses vanilla `org.apache.calcite` classes, it's not a correct way, we need to use reflect to access the Flink runtime shaded calcite classes, and fallback to vanilla one(only for ut.)
   
   > I'm thinking about whether I should fix it in this PR or open another issue and fix it in the future.
   
   Either is fine. Choose the one you like.


-- 
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: notifications-unsubscribe@kyuubi.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org