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

INGEST: Extend KV Processor (#31789) (#32232) #32262

Merged
merged 1 commit into from
Jul 21, 2018

Conversation

original-brownbear
Copy link
Member

Added more capabilities supported by LS to the KV processor:

  • Stripping of brackets and quotes from values (include_brackets in corresponding LS filter)
  • Adding key prefixes
  • Trimming specified chars from keys and values

Refactored the way the filter is configured to avoid conditionals during execution.
Refactored Tests a little to not have to add more redundant getters for new parameters.

Relates #31786

  • Add documentation

Backport of #32232, PR just to run Jenkins

* INGEST: Extend KV Processor (elastic#31789)

Added more capabilities supported by LS to the KV processor:
* Stripping of brackets and quotes from values (`include_brackets` in corresponding LS filter)
* Adding key prefixes
* Trimming specified chars from keys and values

Refactored the way the filter is configured to avoid conditionals during execution.
Refactored Tests a little to not have to add more redundant getters for new parameters.

Relates elastic#31786
* Add documentation
@original-brownbear original-brownbear added >enhancement :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP labels Jul 21, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@original-brownbear original-brownbear merged commit d48b102 into elastic:6.x Jul 21, 2018
@original-brownbear original-brownbear deleted the 32232-6.x branch July 21, 2018 11:40
dnhatn added a commit that referenced this pull request Jul 22, 2018
* 6.x:
  Improve message when JAVA_HOME not set (#32022)
  INGEST: Make a few Processors callable by Painless (#32170) (#32261)
  INGEST: Extend KV Processor (#31789) (#32232) (#32262)
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 >enhancement >non-issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants