xndai opened a new pull request #1884: [CALCITE-2970] Performance issue when enabling abstract converter for EnumerableConvention URL: https://github.com/apache/calcite/pull/1884 During the trait conversion, when the target convention is known, we should create RelNode with corresponding convention instead of adding a logical RelNode. This would reduce a large amount of rule firing and speed up the planning. - In RelBuilderFactory class, we add a new method registerFactoriesWithConvention() to register RelNode factories for a given convention. - In RelBuilder, a new method withConvention() can be used to specifiy the convention while creating RelNodes. - TraitDef convert() method will now use RelBuilder to create physical RelNodes if convention is specified and its RelNode factory is registered - Add new method registerEnumerableRelNodeFactories() for RelOptUtil which register Enumerable RelNode factories into RelBuilderFactory. Currently we only provide factory for EnumerableSort which would be used in RelCollation convert, but it could be extended in the future.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
