TomaYoko commented on PR #28440: URL: https://github.com/apache/doris/pull/28440#issuecomment-1857645489
And whether the test I removed before should be added back( I'm wonder if this will cause a grammer failure again) > @TomaYoko Leave the result file problem, I will fix it. We still have some problem on PipelineX. maybe update `be/src/pipeline/pipeline_x/dependency.h` on line 499 in same way. > > ```c++ > //first:column_id, could point to origin column or cast column > //second:idx mapped to column types > std::unordered_map<int, int> build_col_idx; > ``` > > BTW, PipelineX need branch master. @zhiqiang-hhhh Do I need to make the modification you mentioned together or keep it as it is? And whether the test I removed before should be added back( I'm wonder if this will cause a grammer failure again) -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
