Skip to content

Conversation

V1NAY8
Copy link
Contributor

@V1NAY8 V1NAY8 commented Jul 16, 2021

Closes #1613

@sethmlarson Take a look :)

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@V1NAY8 V1NAY8 force-pushed the reindex_datastream branch from 51d450b to 2962b6b Compare July 17, 2021 13:39
@V1NAY8 V1NAY8 requested a review from sethmlarson July 17, 2021 13:44
Copy link
Contributor

@sethmlarson sethmlarson left a comment

Choose a reason for hiding this comment

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

Thanks for this, I've added a bunch of comments for you:

Copy link
Contributor

@sethmlarson sethmlarson left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!!

@sethmlarson sethmlarson merged commit 83a9198 into elastic:master Aug 2, 2021
@V1NAY8 V1NAY8 deleted the reindex_datastream branch August 2, 2021 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants