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

Update expiration time based on ttl on the source coordinator item. #5262

Merged
merged 2 commits into from
Dec 17, 2024

Conversation

sb2k16
Copy link
Member

@sb2k16 sb2k16 commented Dec 13, 2024

Description

Data Prepper is not updating the expiration time on the source coordination table item when it tries to update the source partition to ASSIGNED. This PR is to add an expiration time on the item based on the ttl.

Check List

  • New functionality includes testing.
  • Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

graytaylor0
graytaylor0 previously approved these changes Dec 13, 2024
@dinujoh
Copy link
Member

dinujoh commented Dec 13, 2024

Build is failing

Error: eckstyle] [ERROR] /home/runner/work/data-prepper/data-prepper/data-prepper-plugins/otel-metrics-source/src/test/java/org/opensearch/dataprepper/plugins/source/otelmetrics/OTelMetricsSourceTest.java:39:8: Unused import - io.opentelemetry.proto.collector.trace.v1.TraceServiceGrpc. [UnusedImports]

Looks like not related to your commit but some other commit broke it.

@sb2k16 sb2k16 force-pushed the dynamo-add-ttl-item branch from fbaa3dc to 634c031 Compare December 13, 2024 20:36
Signed-off-by: Souvik Bose <souvbose@amazon.com>
@sb2k16 sb2k16 force-pushed the dynamo-add-ttl-item branch from 634c031 to 2cb110d Compare December 14, 2024 00:54
Signed-off-by: Souvik Bose <souvbose@amazon.com>
@sb2k16 sb2k16 force-pushed the dynamo-add-ttl-item branch from 1494745 to 839250f Compare December 16, 2024 21:29
@sb2k16 sb2k16 merged commit 8bd39d3 into opensearch-project:main Dec 17, 2024
45 of 47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants