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.

Sign up for GitHub

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

[SQL] Add array constructor test #8727

Merged

akedin merged 1 commit into apache:master from akedin:array-constructor

May 30, 2019

Contributor

akedin commented May 30, 2019

This seems to be missing from the test coverage, asked on Stackoverflow.

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.


          [SQL] Add array constructor test

b74cb17

akedin force-pushed the array-constructor branch from 18165a0 to b74cb17 Compare

May 30, 2019 17:33

Contributor Author

akedin commented May 30, 2019

R: @apilloud @amaliujia

Member

apilloud commented May 30, 2019

LGTM

akedin merged commit e8666a4 into apache:master

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

Labels

None yet