doris [fix](nereids)should use nereids expr's nullable info when call Expr's toThrift method

[fix](nereids)should use nereids expr's nullable info when call Expr's toThrift method

Open starocean999 opened this issue 9 months ago • 6 comments

Proposed changes

when passing the expr's nullable flag to be, the old code call the expr's isNullable method to get this value. The isNullable method is obsolete and may result in wrong value. So we add a new member nullableFromNereids to let nereids to pass the correct nullable info to be

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...