New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SPARK-32021][SQL] Increase precision of seconds and fractions of make_interval
#28873
Conversation
@dongjoon-hyun @juliuszsompolski Please, review this PR. |
Test build #124297 has finished for PR 28873 at commit
|
Thank you for pinging me, @MaxGekk . |
Test build #124301 has finished for PR 28873 at commit
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, LGTM. Thank you, @MaxGekk .
Merged to master.
Could you make a backporting PR to branch-3.0? We need to regenerate some files. |
…ke_interval` Change precision of seconds and its fraction from 8 to 18 to be able to construct intervals of max allowed microseconds value (long). To improve UX of Spark SQL. Yes - Add tests to IntervalExpressionsSuite - Add an example to the `MakeInterval` expression - Add tests to `interval.sql` Closes apache#28873 from MaxGekk/make_interval-sec-precision. Authored-by: Max Gekk <max.gekk@gmail.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
@dongjoon-hyun This is the backport #28878 |
…ke_interval` Change precision of seconds and its fraction from 8 to 18 to be able to construct intervals of max allowed microseconds value (long). To improve UX of Spark SQL. Yes - Add tests to IntervalExpressionsSuite - Add an example to the `MakeInterval` expression - Add tests to `interval.sql` Closes apache#28873 from MaxGekk/make_interval-sec-precision. Authored-by: Max Gekk <max.gekk@gmail.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> [SPARK-31957][SQL] Cleanup hive scratch dir for the developer api startWithContext Comparing to the long-running ThriftServer via start-script, we are more likely to hit the issue https://issues.apache.org/jira/browse/HIVE-10415 / https://issues.apache.org/jira/browse/SPARK-31626 in the developer API `startWithContext` This PR apply SPARK-31626 to the developer API `startWithContext` Fix the issue described in SPARK-31626 Yes, the hive scratch dir will be deleted if cleanup is enabled for calling `startWithContext` new test Closes apache#28784 from yaooqinn/SPARK-31957. Authored-by: Kent Yao <yaooqinn@hotmail.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> [SPARK-32039][YARN][WEBUI][Master] Adding Check to not to set UI port (spark.ui.port) property if mentioned explicitly [SPARK-32039][YARN][WEBUI][Master] Adding Check to not to set UI port (spark.ui.port) property if mentioned explicitly Merge branch 'ahujarajat261/SPARK-32039-change-yarn-webui-port-range-with-property-latest-spark' of https://github.com/rajatahujaatinmobi/spark-1 into ahujarajat261/SPARK-32039-change-yarn-webui-port-range-with-property-latest-spark
What changes were proposed in this pull request?
Change precision of seconds and its fraction from 8 to 18 to be able to construct intervals of max allowed microseconds value (long).
Why are the changes needed?
To improve UX of Spark SQL.
Does this PR introduce any user-facing change?
Yes
How was this patch tested?
MakeInterval
expressioninterval.sql