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

Rename measurementID to measureName in dropMeasurementSeries Method #3400

Open
tenthe opened this issue Jan 3, 2025 · 2 comments
Open

Rename measurementID to measureName in dropMeasurementSeries Method #3400

tenthe opened this issue Jan 3, 2025 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers java Pull requests that update Java code
Milestone

Comments

@tenthe
Copy link
Contributor

tenthe commented Jan 3, 2025

Body

The variable name measurementID in the dropMeasurementSeries method of the DataLakeManagementController class is misleading. It represents measureName, not a measurement ID, causing potential confusion and inconsistency.

StreamPipes Committer

I acknowledge that I am a maintainer/committer of the Apache StreamPipes project.

@tenthe tenthe added enhancement New feature or request good first issue Good for newcomers java Pull requests that update Java code labels Jan 3, 2025
@tenthe tenthe added this to the 0.98.0 milestone Jan 3, 2025
@lezhdz98
Copy link

lezhdz98 commented Jan 7, 2025

I'm new doing contributions to open-source projects and I think this could be a good first task, can you assign this to me?

@tenthe
Copy link
Contributor Author

tenthe commented Jan 8, 2025

Hi @lezhdz98,

Thank you for reaching out to us! This issue is currently being addressed in PR #3401.

We encourage you to explore other open issues with the tag good first issue — your contributions are always welcome

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers java Pull requests that update Java code
Projects
None yet
Development

No branches or pull requests

2 participants