dbatomic commented on PR #44261:
URL: https://github.com/apache/spark/pull/44261#issuecomment-1851896133

   After looking at test failures, there seem to be 'minor' correctness bug for 
time aware expressions that rely on `ReplaceCurrentLike` rule. For following 
expression:
   
   ```sql
   select b from values (
    'one', current_timestamp),
    'two', current_timestamp),
    'three', current_timestamp as data(a, b)
   ```
   
   current_timestamp will not be replaced. Instead, it will be evaluated every 
time which means that these three rows will have different values (tested 
locally). This happens because `ResolveInlineTables` rule happens first and it 
tries to eval all the expressions. IMO, `ReplaceCurrentLike` should have higher 
precedence.
   I will try to fix this as part of this PR.
   
   @cloud-fan  and @MaxGekk as FYI.


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