killbill / killbill-commons

Kill Bill reusable Java components
https://killbill.io
Apache License 2.0
136 stars 67 forks source link

Review batchClaimEntries suspicious log #32

Open pierre opened 6 years ago

pierre commented 6 years ago

Seen in the field:

DBBackedQueue-notif-analytics_notifications: batchClaimEntries only claimed partial entries 100/98

Why can this still happen?

pierre commented 5 years ago

While I don't fully understand the issue, https://github.com/killbill/killbill-commons/pull/70 most likely works around it (get + claim done in one transaction).

Will close after verifying we don't see these anymore in production logs.