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

Unmute and fix testSubParserArray #40626

Merged
merged 1 commit into from
Mar 29, 2019

Conversation

henningandersen
Copy link
Contributor

testSubParserArray failed, fixed and improved to not always have an
object as outer-level inside array.

Closes #40617

testSubParserArray failed, fixed and improved to not always have an
object as outer-level inside array.

Closes elastic#40617
@henningandersen henningandersen added >test Issues or PRs that are addressing/adding tests :Analytics/Geo Indexing, search aggregations of geo points and shapes v8.0.0 labels Mar 29, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo

Copy link
Member

@cbuescher cbuescher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@henningandersen
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/2

Copy link
Contributor

@imotov imotov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM2. Thanks!

@henningandersen henningandersen merged commit 3fcfd60 into elastic:master Mar 29, 2019
henningandersen added a commit that referenced this pull request Mar 29, 2019
testSubParserArray failed, fixed and improved to not always have an
object as outer-level inside array.

Closes #40617
henningandersen added a commit that referenced this pull request Mar 29, 2019
testSubParserArray failed, fixed and improved to not always have an
object as outer-level inside array.

Closes #40617
@henningandersen
Copy link
Contributor Author

henningandersen commented Mar 29, 2019

Backported to 7.x, 7.0 and 6.7

jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Apr 1, 2019
* elastic/7.0:
  [TEST] Mute WebhookHttpsIntegrationTests.testHttps
  [DOCS] Add 'time value' links to several monitor settings (elastic#40633) (elastic#40687)
  Do not perform cleanup if Manifest write fails with dirty exception (elastic#40519)
  Remove mention of soft deletes from getting started (elastic#40668)
  Fix bug in detecting use of bundled JDK on macOS
  Reindex conflicts clarification (docs) (elastic#40442)
  SQL: [Tests] Enable integration tests for fixed issues (elastic#40664)
  Add information about the default sort mode (elastic#40657)
  SQL: [Docs] Fix example for CURDATE
  SQL: [Docs] Fix doc errors regarding CURRENT_DATE. (elastic#40649)
  Clarify using time_zone and date math in range query (elastic#40655)
  Add notice for bundled jdk (elastic#40576)
  disable kerberos test until kerberos fixture is working again
  [DOCS] Use "source" instead of "inline" in ML docs (elastic#40635)
  Unmute and fix testSubParserArray (elastic#40626)
  Geo Point parse error fix (elastic#40447)
  Increase suite timeout to 30 minutes for docs tests (elastic#40521)
  Fix repository-hdfs when no docker and unnecesary fixture
  Avoid building hdfs-fixure use an image that works instead
henningandersen added a commit that referenced this pull request Apr 8, 2019
testSubParserArray failed, fixed and improved to not always have an
object as outer-level inside array.

Closes #40617
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes >test Issues or PRs that are addressing/adding tests v6.7.1 v7.0.0-rc2 v7.2.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] XContentParserTests#testSubParserArray fails frequently
5 participants