Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clickhouse backend broadcast relation support r2c #888

Closed
loneylee opened this issue Jan 29, 2023 · 0 comments
Closed

Clickhouse backend broadcast relation support r2c #888

loneylee opened this issue Jan 29, 2023 · 0 comments
Labels
clickhouse backend works for ClickHouse backend enhancement New feature or request

Comments

@loneylee
Copy link
Member

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Test case GlutenSubquerySuite.'correlated scalar subquery in where'

CHNativeColumnarToRow
+- ProjectExecTransformer [_1#220 AS a#225, _2#221 AS b#226]
   +- CHBroadcastHashJoinExecTransformer [_1#220], [c#236], Inner, BuildLeft, (_2#221 < max(d)#242), false
      :- BroadcastQueryStage 1
      :  +- ColumnarBroadcastExchange HashedRelationBroadcastMode(List(cast(input[0, int, true] as bigint)),false), [plan_id=394]
      :     +- RowToCHNativeColumnar
      :        +- LocalTableScan [_1#220, _2#221]
      +- FilterExecTransformer isnotnull(max(d)#242)
         +- HashAggregateTransformer(keys=[c#236], functions=[max(d#237)], output=[max(d)#242, c#236])
            +- ShuffleQueryStage 0
               +- ColumnarExchangeAdaptor hashpartitioning(c#236, 1), ENSURE_REQUIREMENTS, false, [plan_id=350], [id=#350], [OUTPUT] List(c:IntegerType, max:DoubleType), [OUTPUT] List(c:IntegerType, max:DoubleType)
                  +- *(4)HashAggregateTransformer(keys=[c#236], functions=[partial_max(d#237)], output=[c#236, max#246])
                     +- ProjectExecTransformer [_1#231 AS c#236, _2#232 AS d#237]
                        +-FilterExecTransformer isnotnull(_1#231)
                           +- RowToCHNativeColumnar
                              +- LocalTableScan [_1#231, _2#232]

ColumnarBroadcastExchange broadcast relation not suport RowToCHNativeColumnar

Describe the solution you'd like
add support
case RowToCHNativeColumnarExec: xxx

@loneylee loneylee added the enhancement New feature or request label Jan 29, 2023
@zzcclp zzcclp closed this as completed Feb 2, 2023
@weiting-chen weiting-chen added the clickhouse backend works for ClickHouse backend label Apr 2, 2023
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clickhouse backend works for ClickHouse backend enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants