-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Add array support using definitions in the _meta field. #2462
Add array support using definitions in the _meta field. #2462
Conversation
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Brian Olsen.
|
Just submitted my cla agreement. |
Not sure why but a hive test failed when all my changes were in presto-elasticsearch. Are these typical issues with the build or should I investigate this more? |
No, those are unrelated. There are some flaky tests we're trying to track down. |
@cla-bot check |
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Brian Olsen.
|
The cla-bot has been summoned, and re-checked this pull request! |
hmm will I need to recommit after setting my email in the config? |
I was missing my credential information when submitting my commit. I just created an identical branch and resubmitted with the appropriate info and opened this PR. #2478 |
Still needs a few more tests and discussion.
Fixes: #2441