Skip to content
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

Use HALF_EVEN rounding for the double type in generated code #142

Closed

Conversation

mbautin
Copy link

@mbautin mbautin commented Jan 12, 2016

@markhamstra
Copy link

This looks good, but the round test in catalyst/expressions/MathFunctionsSuite.scala needs to be updated.

@mbautin
Copy link
Author

mbautin commented Jan 12, 2016

@markhamstra: updated.

@mbautin mbautin closed this Feb 1, 2016
markhamstra pushed a commit to markhamstra/spark that referenced this pull request Sep 12, 2016
Upgrades the Snappy version to 1.1.2.6 from 1.1.2.4, release notes: https://github.com/xerial/snappy-java/blob/master/Milestone.md mention "Fix a bug in SnappyInputStream when reading compressed data that happened to have the same first byte with the stream magic header (alteryx#142)"

Existing unit tests using the latest IBM Java 8 on Intel, Power and Z architectures (little and big-endian)

Author: Adam Roberts <[email protected]>

Closes apache#14958 from a-roberts/master.

(cherry picked from commit 6c08dbf)
Signed-off-by: Sean Owen <[email protected]>
markhamstra pushed a commit to markhamstra/spark that referenced this pull request Sep 26, 2016
## What changes were proposed in this pull request?

Upgrades the Snappy version to 1.1.2.6 from 1.1.2.4, release notes: https://github.com/xerial/snappy-java/blob/master/Milestone.md mention "Fix a bug in SnappyInputStream when reading compressed data that happened to have the same first byte with the stream magic header (alteryx#142)"

## How was this patch tested?
Existing unit tests using the latest IBM Java 8 on Intel, Power and Z architectures (little and big-endian)

Author: Adam Roberts <[email protected]>

Closes apache#14958 from a-roberts/master.
markhamstra pushed a commit to markhamstra/spark that referenced this pull request Nov 7, 2017
* Update client version

* Upgrade minikube

* Update pom.xml
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.

2 participants