Skip to content

[Kafka] Remove unused mapping fields#15225

Merged
agithomas merged 4 commits intoelastic:mainfrom
agithomas:issue-388-jvm-unused-fieldmappings
Sep 9, 2025
Merged

[Kafka] Remove unused mapping fields#15225
agithomas merged 4 commits intoelastic:mainfrom
agithomas:issue-388-jvm-unused-fieldmappings

Conversation

@agithomas
Copy link
Contributor

@agithomas agithomas commented Sep 8, 2025

  • Bug

Proposed commit message

Remove unused mapping fields from the field mappings

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • Package upgrade testing

How to test this PR locally

  • elastic-package build && elastic-package stack up -v -d --services package-registry

@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Sep 8, 2025

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

elasticmachine commented Sep 8, 2025

💚 Build Succeeded

@elastic-sonarqube
Copy link

elastic-sonarqube bot commented Sep 8, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

@andrewkroh andrewkroh added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:kafka Kafka labels Sep 8, 2025
@agithomas agithomas marked this pull request as ready for review September 9, 2025 05:21
@agithomas agithomas requested a review from a team as a code owner September 9, 2025 05:21
ishleenk17
ishleenk17 reviewed Sep 9, 2025
View reviewed changes
@agithomas agithomas merged commit 252f28a into elastic:main Sep 9, 2025
9 checks passed
@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Sep 9, 2025

Package kafka - 1.21.1 containing this change is available at https://epr.elastic.co/package/kafka/1.21.1/

tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025
@agithomas
[Kafka] Remove unused mapping fields (elastic#15225)
a501988 
* Remove unused field mappings for the jvm dataset in Kafka package

* Removed unused mapping fiel mappings

* Removed commented field mappings

* Updated PR link
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ishleenk17 ishleenk17 ishleenk17 approved these changes

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:kafka Kafka

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@agithomas @elasticmachine @ishleenk17 @andrewkroh