11/23/2013 1:48:15 PM
the magic and add tests for eventsStreamBuilder.buildForAccount. Even though
all tests were passing, I still went ahead and updated buildForAccount to do the same
magic as buildForEntitlement. I didn't manage to write a failing test scenario for it
but it seems required.
This also fixes an NPE with standalone subscriptions.
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>
|
11/22/2013 8:39:38 PM
in one class and simplify the code
|
11/21/2013 11:38:41 PM
Pierre-Alexandre Meyer <pierre@mouraf.org>
|
11/21/2013 10:19:50 PM
Pierre-Alexandre Meyer <pierre@mouraf.org>
|
11/21/2013 10:08:01 PM
by limiting the amount of database calls.
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>
|
11/27/2013 12:47:36 PM
patch removes getBlockingAll(UUID blockableId, BlockingStateType blockingStateType, InternalTenantContext context);
from BlockingInternalApi: junction was calling it multiple times for different blockingStateType, which
was breaking the natural ordering of events and hence introducing bugs when computing blocking durations.
Instead, we introduce getBlockingAllForAccount(InternalTenantContext context);
Events are ordered correctly by junction and will return bundle, subscription and account level blocking
events (we were ignoring the subscription level ones before for some reason).
This also:
* fixes the ordering returned by the SQL statements (only really needed for getBlockingStateForService
as we re-order the other calls in ProxyBlockingStateDao).
* removes BillingEventSet#isLast which wasn't used.
This closes #123.
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>
|
|
|
|
|
11/20/2013 11:25:08 AM
had a report that this method would pick up soft deleted
tags but I cannot reproduce it.
While we're at it, get rid of the unnecessary accountId parameter
to the method.
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>
|
11/20/2013 10:57:34 AM
Meyer <pierre@mouraf.org>
|
