|
|
11/25/2013 3:56:37 PM
Pierre-Alexandre Meyer <pierre@mouraf.org>
|
11/25/2013 12:49:57 PM
the getSubscriptionBundlesForAccountId API (which is
behind /1.0/kb/accounts/ACCOUNT_ID/bundles) and optimize the various
layers to limit the amount of DAO calls.
1.0 call timings for an account with 35 bundles and 67 subscriptions:
* Before changes: 8m40s
* After changes: 6.50s at startup, 4.20s after caches are warm
Since this call should be fast, change most APIs in DefaultSubscriptionApi
to always build all subscriptions for a given account first, and then do
further filtering in memory (per bundle, etc.). This simplifies the amount
of codepaths to optimize.
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>
|
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/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/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>
|
