OIiveirra commented on code in PR #69032:
URL: https://github.com/apache/doris/pull/69032#discussion_r4388316138


##########
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/BindExpression.java:
##########
@@ -304,6 +306,10 @@ private LogicalExternalRowLevelMergeSink<Plan> 
bindIcebergMergeSink(
             validateMergeNoLossyCoercion(sink.getCols(), sink.child(), 
ctx.cascadesContext.getCteContext());
         }
         List<NamedExpression> castExprs = coerceMergeOutput(sink.getCols(), 
outputExprs);

Review Comment:
   Fixed in fcd19869858. Reused proven-NULL normalization before row-level sink 
coercion, aligned to data output positions. Added Geometry/Geography typed-NULL 
UPDATE and routing/passthrough/ordinary-cast controls; non-NULL identity checks 
remain.



-- 
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]

Reply via email to