Skip to content

Conversation

@olaky
Copy link
Contributor

@olaky olaky commented Jun 9, 2022

What changes were proposed in this pull request?

Apply the optimizer rule ComputeCurrentTime consistently across subqueries.

This is a backport of #36654 with adjustements:

  • The rule does not use pruning
  • The transformWithSubqueries function was also backported

Why are the changes needed?

At the moment timestamp functions like now() can return different values within a query if subqueries are involved

Does this PR introduce any user-facing change?

No

How was this patch tested?

A new unit test was added

@olaky olaky changed the base branch from master to branch-3.0 June 9, 2022 11:08
@github-actions github-actions bot added the SQL label Jun 9, 2022
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@dongjoon-hyun
Copy link
Member

dongjoon-hyun commented Jun 11, 2022

Thank you for making a backporting PR, but branch-3.0 is EOL currently because Apache Spark 3.0.0 was two years ago.

According to the Apache Spark community policy, I close this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants