Skip to content

Conversation

@abradbury
Copy link
Contributor

What changes were proposed in this pull request?

Improved the documentation for the datetime functions in org.apache.spark.sql.functions by adding details about the supported column input types, the column return type, behaviour on invalid input, supporting examples and clarifications.

How was this patch tested?

Manually testing each of the datetime functions with different input to ensure that the corresponding Javadoc/Scaladoc matches the behaviour of the function. Successfully ran the unidoc SBT process.

@HyukjinKwon
Copy link
Member

ok to test

@HyukjinKwon
Copy link
Member

mind resolving the conflicts?

@abradbury
Copy link
Contributor Author

@HyukjinKwon, merge conflicts have now been resolved

@SparkQA
Copy link

SparkQA commented Jun 9, 2018

Test build #91606 has finished for PR 20901 at commit 6dea103.

  • This patch fails Spark unit tests.
  • This patch does not merge cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jun 9, 2018

Test build #91620 has finished for PR 20901 at commit 2544d93.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 25, 2018

Test build #4291 has finished for PR 20901 at commit 2544d93.

  • This patch fails Java style tests.
  • This patch does not merge cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Aug 25, 2018

The failure isn't related:

[error] (kubernetes-integration-tests/*:checkstyle) java.io.FileNotFoundException: checkstyle-config.xml (No such file or directory)
[error] Total time: 258 s, completed Aug 25, 2018 2:45:08 PM

(There has never been a checkstyle-config.xml in the project)
Master test jobs don't hit this, but a few other PR builder jobs do. Some don't. Weird, I'll try again.

@SparkQA
Copy link

SparkQA commented Aug 26, 2018

Test build #4292 has finished for PR 20901 at commit 2544d93.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Aug 26, 2018

Merged to master

@asfgit asfgit closed this in ad43e2c Aug 26, 2018
bogdanrdc pushed a commit to bogdanrdc/spark that referenced this pull request Aug 28, 2018
## What changes were proposed in this pull request?

Improved the documentation for the datetime functions in `org.apache.spark.sql.functions` by adding details about the supported column input types, the column return type, behaviour on invalid input, supporting examples and clarifications.

## How was this patch tested?

Manually testing each of the datetime functions with different input to ensure that the corresponding Javadoc/Scaladoc matches the behaviour of the function. Successfully ran the `unidoc` SBT process.

Closes apache#20901 from abradbury/SPARK-23792.

Authored-by: Adam Bradbury <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants