Skip to content

Conversation

@davies
Copy link
Contributor

@davies davies commented Jul 10, 2015

It may loss precision in microseconds when using float for it.

@SparkQA
Copy link

SparkQA commented Jul 10, 2015

Test build #1040 has finished for PR 7344 at commit 249ec61.

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

davies pushed a commit that referenced this pull request Jul 10, 2015
It may loss precision in microseconds when using float for it.

Author: Davies Liu <[email protected]>

Closes #7344 from davies/fix_date_test and squashes the following commits:

249ec61 [Davies Liu] fix flaky test
Copy link
Contributor

Choose a reason for hiding this comment

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

can you add an inline comment explaining what's happening here?

Copy link
Contributor

Choose a reason for hiding this comment

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

(and just merge it after adding the comment)

@davies
Copy link
Contributor Author

davies commented Jul 10, 2015

It was merged already.

@davies davies closed this Jul 10, 2015
@rxin
Copy link
Contributor

rxin commented Jul 10, 2015

OK can you push a commit directly to master to add inline comment?

@davies
Copy link
Contributor Author

davies commented Jul 11, 2015

Added. b6fc0ad

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.

3 participants