Skip to content

add @Override in EnrichPlugin extend method#76873

Merged
martijnvg merged 2 commits intoelastic:masterfrom
weizijun:fix-EnrichPlugh-miss-Override
Oct 11, 2021
Merged

add @Override in EnrichPlugin extend method#76873
martijnvg merged 2 commits intoelastic:masterfrom
weizijun:fix-EnrichPlugh-miss-Override

Conversation

@weizijun
Copy link
Copy Markdown
Contributor

Same as title.
add @OverRide in EnrichPlugin extend method.

@cla-checker-service
Copy link
Copy Markdown

cla-checker-service bot commented Aug 24, 2021

💚 CLA has been signed

@elasticsearchmachine elasticsearchmachine added v8.0.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Aug 24, 2021
@weizijun weizijun force-pushed the fix-EnrichPlugh-miss-Override branch from 7d9fc0e to 3fb2019 Compare August 24, 2021 12:18
@jtibshirani jtibshirani added the :Distributed/Ingest Node Execution or management of Ingest Pipelines label Oct 6, 2021
@elasticmachine elasticmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label Oct 6, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@martijnvg
Copy link
Copy Markdown
Member

Thanks @weizijun, this looks good! I will merge it soon.

@martijnvg
Copy link
Copy Markdown
Member

@elasticmachine test this please

@weizijun
Copy link
Copy Markdown
Contributor Author

weizijun commented Oct 7, 2021

@elasticmachine update branch

@weizijun
Copy link
Copy Markdown
Contributor Author

weizijun commented Oct 8, 2021

@elasticmachine test this please

@martijnvg
Copy link
Copy Markdown
Member

@elasticmachine ok to test

@martijnvg martijnvg merged commit 5f737a9 into elastic:master Oct 11, 2021
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Oct 11, 2021
* upstream/master: (250 commits)
  [Transform] HLRC cleanups (elastic#78909)
  [ML] Make ML indices hidden when the node becomes master (elastic#77416)
  Introduce a Few Settings Singleton Instances (elastic#78897)
  Simplify TestCluster extraJar configuration (elastic#78837)
  Add @OverRide annotations to methods in EnrichPlugin class (elastic#76873)
  Add v7 restCompat for invalidating API key with the id field (elastic#78664)
  EQL: Refine repeatable queries (elastic#78895)
  Fix DataTierTests package and add a validation test (elastic#78880)
  Fix split package org.elasticsearch.common.xcontent (elastic#78831)
  Store DataTier Preference directly on IndexMetadata (elastic#78668)
  [DOCS] Fixes typo in calendar API example (elastic#78867)
  Improve Node Shutdown Observability (elastic#78727)
  Convert encrypted snapshot license object to LicensedFeature (elastic#78731)
  Revert "Make nodePaths() singular (elastic#72514)" (elastic#78801)
  Fix incorrect generic type in PolicyStepsRegistry (elastic#78628)
  [DOCS] Fixes ML get calendars API (elastic#78808)
  Implement GET API for System Feature Upgrades (elastic#78642)
  [TEST] More MetadataStateFormat tests (elastic#78577)
  Add support for rest compatibility headers to the HLRC (elastic#78490)
  Un-ignoring tests after backporting fix (elastic#78830)
  ...

# Conflicts:
#	server/src/main/java/org/elasticsearch/ingest/IngestService.java
#	server/src/test/java/org/elasticsearch/ingest/IngestServiceTests.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Ingest Node Execution or management of Ingest Pipelines external-contributor Pull request authored by a developer outside the Elasticsearch team >non-issue Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v7.16.0 v8.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants