zhiqiang-hhhh commented on PR #28440: URL: https://github.com/apache/doris/pull/28440#issuecomment-1857696470
> > @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 wonder if this will cause a grammer failure again) Just make it together and add the test case back. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
