Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add target_field parameter to ingest processors #24133

Merged
merged 1 commit into from Jun 13, 2017

Conversation

Projects
None yet
6 participants
@alexshadow007
Copy link
Contributor

commented Apr 17, 2017

Parameter was added to gsub, join, lowercase, sort, split, trim and uppercase processors.

Closes #23682 #23228

@elasticmachine

This comment has been minimized.

Copy link
Collaborator

commented Apr 17, 2017

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?

@talevy

This comment has been minimized.

Copy link
Contributor

commented Jun 9, 2017

Hi @alexshadow007, these changes look good! thank you so much. Apologies for the delayed response here. Do you mind rebasing off latest master to resolve the conflicting files?

Jenkins, please test this

@clintongormley clintongormley added v5.6.0 and removed v5.5.0 labels Jun 12, 2017

@alexshadow007 alexshadow007 force-pushed the alexshadow007:23682/target_field branch to 43d6720 Jun 12, 2017

@alexshadow007

This comment has been minimized.

Copy link
Contributor Author

commented Jun 12, 2017

@talevy Done

@talevy

This comment has been minimized.

Copy link
Contributor

commented Jun 12, 2017

thanks!

jenkins, please test this

@talevy talevy merged commit a7dafda into elastic:master Jun 13, 2017

2 checks passed

CLA Commit author has signed the CLA
Details
elasticsearch-ci Build finished.
Details

talevy added a commit that referenced this pull request Jun 13, 2017

@talevy

This comment has been minimized.

Copy link
Contributor

commented Jun 13, 2017

Merged into master and 5.x for the 6.0 and 5.6 releases!

jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Jun 14, 2017

Merge branch 'master' into primary-context
* master: (27 commits)
  Refactor TransportShardBulkAction.executeUpdateRequest and add tests
  Make sure range queries are correctly profiled. (elastic#25108)
  Test: allow setting socket timeout for rest client (elastic#25221)
  Migration docs for elastic#25080 (elastic#25218)
  Remove `discovery.type` BWC layer from the EC2/Azure/GCE plugins elastic#25080
  When stopping via systemd only kill the JVM, not its control group (elastic#25195)
  Remove PrefixAnalyzer, because it is no longer used.
  Internal: Remove Strings.cleanPath (elastic#25209)
  Docs: Add note about which secure settings are valid (elastic#25212)
  Indices.rollover/10_basic should refresh to make the doc visible in lucene stats
  Port support for commercial GeoIP2 databases from Logstash. (elastic#24889)
  [DOCS] Add ML node to node.asciidoc (elastic#24495)
  expose simple pattern tokenizers (elastic#25159)
  Test: add setting to change request timeout for rest client (elastic#25201)
  Fix secure repository-hdfs tests on JDK 9
  Add target_field parameter to gsub, join, lowercase, sort, split, trim, uppercase (elastic#24133)
  Add Cross Cluster Search support for scroll searches (elastic#25094)
  Adapt skip version in rest-api-spec/test/indices.rollover/20_max_doc_condition.yml
  Rollover max docs should only count primaries (elastic#24977)
  Add remote cluster infrastructure to fetch discovery nodes. (elastic#25123)
  ...

@colings86 colings86 added v6.0.0-beta1 and removed v6.0.0 labels Jul 31, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.