Skip to content

Conversation

@BryanCutler
Copy link
Member

Python time values return a floating point value, need to cast to integer before serialize with struct.pack('!q', value)

https://issues.apache.org/jira/browse/SPARK-6931

@JoshRosen
Copy link
Contributor

Jenkins, this is ok to test.

@SparkQA
Copy link

SparkQA commented Sep 4, 2015

Test build #41987 has finished for PR 8594 at commit feed499.

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

@BryanCutler
Copy link
Member Author

Looks like the style, tests, and mima passed, I'm not sure why it was marked as Failed..

@rxin
Copy link
Contributor

rxin commented Sep 5, 2015

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Sep 5, 2015

Test build #42049 has finished for PR 8594 at commit feed499.

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

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42049/
Test FAILed.

@davies
Copy link
Contributor

davies commented Sep 10, 2015

LGTM, merging into 1.2 and 1.3, thanks!

asfgit pushed a commit that referenced this pull request Sep 10, 2015
…rialization

Python time values return a floating point value, need to cast to integer before serialize with struct.pack('!q', value)

https://issues.apache.org/jira/browse/SPARK-6931

Author: Bryan Cutler <[email protected]>

Closes #8594 from BryanCutler/py-write_long-backport-6931-1.2.
asfgit pushed a commit that referenced this pull request Sep 10, 2015
…rialization

Python time values return a floating point value, need to cast to integer before serialize with struct.pack('!q', value)

https://issues.apache.org/jira/browse/SPARK-6931

Author: Bryan Cutler <[email protected]>

Closes #8594 from BryanCutler/py-write_long-backport-6931-1.2.

(cherry picked from commit 4862a80)
Signed-off-by: Davies Liu <[email protected]>
@davies
Copy link
Contributor

davies commented Sep 10, 2015

@BryanCutler This is not get closed automatically, could you close it?

@BryanCutler BryanCutler deleted the py-write_long-backport-6931-1.2 branch October 29, 2015 21:34
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.

6 participants