Skip to content

Conversation

@romseygeek
Copy link
Contributor

Relates to #41059

@romseygeek romseygeek added :Search/Search Search-related issues that do not fall into other categories >breaking-java >refactoring v8.0.0 labels Sep 20, 2019
@romseygeek romseygeek self-assigned this Sep 20, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

@romseygeek
Copy link
Contributor Author

Note: this applies on top of #46926

@jpountz jpountz mentioned this pull request Sep 20, 2019
66 tasks
Copy link
Contributor

@colings86 colings86 left a comment

Choose a reason for hiding this comment

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

LGTM

logger.debug("setting up [{}] docs", numDocs);
indexRandom(true, false, true, IntStream.range(0, numDocs)
.mapToObj(i -> client().prepareIndex(INDEX, TYPE, String.valueOf(i)).setSource("n", i))
.mapToObj(i -> client().prepareIndex(INDEX, "_doc", String.valueOf(i)).setSource("n", i))
Copy link
Contributor

Choose a reason for hiding this comment

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

I assume this is done because we don't yet have a typeless indexing method?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct

@romseygeek
Copy link
Contributor Author

@elasticmachine update branch

@romseygeek
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/bwc

@romseygeek
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/2

@romseygeek romseygeek merged commit c1f99e2 into elastic:master Sep 23, 2019
@romseygeek romseygeek deleted the types-removal/search-response branch September 23, 2019 18:14
@pgomulka pgomulka mentioned this pull request Mar 25, 2020
66 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>breaking-java >refactoring :Search/Search Search-related issues that do not fall into other categories v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants