chunwei comments

Results 10 comments of


                                            chunwei

[CALCITE-4378] Included ProjectJoinTransposeRule into the BASE rules (Jiatao Tao)

The CI failed. Could you resolve it?

[CALCITE-2826] Add versions of create() for LogicalSort and LogicalEx…

Why do you need these methods?

[CALCITE-3351] [WIP]In JDBC adapter, when generating SQL for MYSQL, MySQLSyntaxErrorException throws when there is non-ascii literal

The title is not clear. Could you change it?

[CALCITE-1581] Support UDTF like Hive

This PR comes from https://github.com/apache/calcite/pull/2092/ with some fixups. Besides, It is already reviewed by some members. I think it is good overall.

[CALCITE-4340] Fixing Subqueries in On Clauses

FYI: the commit message should be [CALCITE-4340] xxxxx

[CALCITE-2772] Support varargs UDF for scalar function

@gr4ve, do you still want to work on this? If no, I would like to take over if you don't mind.

CALCITE-3770, minor change "EnumerableCalcRel" appears in code to "EnumerableCalc".

Maybe we can merge it with another pr.

CALCITE-3770, minor change "EnumerableCalcRel" appears in code to "EnumerableCalc".

> > Maybe we can merge it with another pr. > > which PR? PR which is not trivial such as bug fix or a feature.

[CALCITE-4803] AggregateProjectMergeRule lost alias after remove the …

> Before merge this PR. I think maybe we should reach an agreement in Jira. Because this will be exists confuse in the future.(The code implement is different about the...

[CALCITE-4808] Adding assertion to RelOptUtil.getVariablesUsed

Could you merge the commits to one and rebase the PR, @jamesstarr ?