Skip to content

Conversation

@tveasey
Copy link
Contributor

@tveasey tveasey commented Jan 7, 2019

We're going to round trip hashes of the Elasticsearch document identifiers to sanity check the join of analysis results with existing data frame row documents. This makes the necessary changes for doing this in the data_frame_analyzer command.

Copy link
Contributor

@dimitris-athanasiou dimitris-athanasiou left a comment

Choose a reason for hiding this comment

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

LGTM

@tveasey tveasey merged commit 96bcb67 into elastic:feature/analysis-pipeline Jan 9, 2019
@tveasey tveasey deleted the elastic-doc-ids branch May 1, 2019 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants