Skip to content

Conversation

@yaooqinn
Copy link
Member

@yaooqinn yaooqinn commented Oct 31, 2019

What changes were proposed in this pull request?

support interval */ numeric and numeric * interval

Why are the changes needed?

Part of SPARK-27764 Feature Parity between PostgreSQL and Spark

Does this PR introduce any user-facing change?

Yes. add interval '1 second' */ 3.0 support

How was this patch tested?

add ut

Copy link
Member

@MaxGekk MaxGekk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yaooqinn Look at #26132

@yaooqinn
Copy link
Member Author

Oops... Didn't notice that..

@SparkQA
Copy link

SparkQA commented Oct 31, 2019

Test build #113019 has finished for PR 26345 at commit 4ea79bd.

  • This patch fails SparkR unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
  • case class IntervalDivide(left: Expression, right: Expression)
  • case class IntervalMultiply(left: Expression, right: Expression)

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.

4 participants