Skip to content

Conversation

@dadoonet
Copy link
Contributor

@dadoonet dadoonet commented Feb 3, 2017

Let's make our life easier when debugging/testing.
Also having a flat dir helps us to compare or "synchronize" more easily with Tika project files.

Closes #22958.

Let's make our life easier when debugging/testing.
Also having a flat dir helps us to compare or "synchronize" more easily with Tika project files.

Closes elastic#22958.
@dadoonet dadoonet added :Plugin Ingest Attachment >test Issues or PRs that are addressing/adding tests v5.3.0 v6.0.0-alpha1 labels Feb 3, 2017
@dadoonet dadoonet self-assigned this Feb 3, 2017
@dadoonet dadoonet requested a review from nik9000 February 3, 2017 14:21
@dadoonet
Copy link
Contributor Author

dadoonet commented Feb 3, 2017

@nik9000 I'll do another similar PR for mapper attachments plugin in 5.x branch.

@nik9000
Copy link
Member

nik9000 commented Feb 3, 2017

I like this idea because if we change one file we don't have to rewrite the whole zip.... Will review.

@dadoonet
Copy link
Contributor Author

dadoonet commented Feb 3, 2017

Thanks @nik9000

@dadoonet dadoonet merged commit 7a8680c into elastic:master Feb 3, 2017
@dadoonet dadoonet deleted the fix/22958-tika-files-zip branch February 3, 2017 15:14
dadoonet added a commit that referenced this pull request Feb 3, 2017
Let's make our life easier when debugging/testing.
Also having a flat dir helps us to compare or "synchronize" more easily with Tika project files.

Closes #22958.

Backport of #22959 in 5.x branch
dadoonet added a commit to dadoonet/elasticsearch that referenced this pull request Feb 3, 2017
Let's make our life easier when debugging/testing.
Also having a flat dir helps us to compare or "synchronize" more easily with Tika project files.

Related to elastic#22958.
Same as of elastic#22959 but for mapper-attachments plugin
dadoonet added a commit that referenced this pull request Feb 20, 2017
Let's make our life easier when debugging/testing.
Also having a flat dir helps us to compare or "synchronize" more easily with Tika project files.

Closes #22958.

Backport of #22959 in 5.2 branch
dadoonet added a commit that referenced this pull request Feb 20, 2017
Let's make our life easier when debugging/testing.
Also having a flat dir helps us to compare or "synchronize" more easily with Tika project files.

Related to #22958.
Same as of #22959 but for mapper-attachments plugin

Backport of #22963 in 5.2 branch
@clintongormley clintongormley added :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP and removed :Plugin Ingest Attachment labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >test Issues or PRs that are addressing/adding tests v5.2.2 v5.3.0 v6.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants