Skip to content

Conversation

@Mpdreamz
Copy link
Member

…ex() helper, also made sure we use a _doc sort on the scroll

@Mpdreamz Mpdreamz force-pushed the fix/reindex-helper-improvements branch from 5318315 to 3a64d95 Compare November 30, 2016 14:43
Copy link
Contributor

@gmarz gmarz left a comment

Choose a reason for hiding this comment

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

LGTM 👍 My only comment is to maybe rename OmitCreateIndex to OmitIndexCreation.

…ex() helper, also made sure we use a _doc sort on the scroll
@Mpdreamz Mpdreamz force-pushed the fix/reindex-helper-improvements branch from 3a64d95 to 49f24cd Compare November 30, 2016 14:56
@Mpdreamz
Copy link
Member Author

++ to name change added that in as well, thanks @gmarz.

@Mpdreamz Mpdreamz merged commit 54a6ab9 into 5.x Nov 30, 2016
Mpdreamz added a commit that referenced this pull request Nov 30, 2016
…ex() helper, also made sure we use a _doc sort on the scroll (#2434)
Mpdreamz added a commit that referenced this pull request Nov 30, 2016
@Mpdreamz Mpdreamz deleted the fix/reindex-helper-improvements branch November 30, 2016 16:58
Mpdreamz added a commit that referenced this pull request Nov 30, 2016
awelburn pushed a commit to Artesian/elasticsearch-net that referenced this pull request Nov 6, 2017
…r Reindex() helper, also made sure we use a _doc sort on the scroll (elastic#2434)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants