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

Jump to bottom

[BEAM-7529] Add Sums.ofFloats() and Sums.ofDoubles() #8820

Merged

dmvk merged 1 commit into apache:master from je-ik:BEAM-7529

Jun 12, 2019

Contributor

je-ik commented Jun 11, 2019

Fix [BEAM-7529].

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

Choose reviewer(s) and mention them in a comment (R: @username).
Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang	SDK	Apex	Dataflow	Flink	Gearpump	Samza	Spark
Go		---	---		---	---
Java
Python		---			---	---

Pre-Commit Tests Status (on master branch)

---	Java	Python	Go	Website
Non-portable
Portable	---		---	---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.


          [BEAM-7529] Add Sums.ofFloats() and Sums.ofDoubles()

c603563

Contributor Author

je-ik commented Jun 11, 2019

@dmvk Will you have a look at this please?

dmvk self-requested a review

June 12, 2019 20:25

dmvk approved these changes

View reviewed changes

dmvk merged commit d754094 into apache:master

je-ik deleted the BEAM-7529 branch

June 12, 2019 20:35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment