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.

Sign up for GitHub

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

Jump to bottom

fix null pointer exception in getOldestRecordTimeMillis #596

Merged

chenylee-aws merged 1 commit into awslabs:master from jtaub:bugfix-489

Oct 5, 2024

Contributor

jtaub commented Oct 5, 2024

Issue #, if available: 489

Description of changes:

Refactored getOldestRecordTimeInMillis to avoid a possible NullPointerException which can be caused by a race condition
Added a simple unit test for this method

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.


          fix null pointer exception in getOldestRecordTimeMillis

835d54b

chenylee-aws approved these changes

View reviewed changes

Contributor

chenylee-aws left a comment

Thanks for the contribution!

chenylee-aws merged commit 0eafc8d into awslabs:master

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

Labels

None yet