[aerogear-dev] Help needed on AGPUSH-848

[Daniel Bevenius]

Hey Seb,

sure let me take a closer look at this. I'm getting the feeling that it
might not be as simple as that. Let me push something and we can discuss it.




On 31 July 2014 10:39, Sebastien Blanc <scm.blanc at gmail.com> wrote:

> Hi Dan,
> Not sure if I understand exactly what you meant, could do a small snippet
> ?
>
>
>
> On Thu, Jul 31, 2014 at 10:08 AM, Daniel Bevenius <
> daniel.bevenius at gmail.com> wrote:
>
>> Oh I see. Then I'd say you'll need to change the return type to either
>> use a custom object for the key in the map, or perhaps return a list with
>> that came custom object. What ever makes the most sense in this use case.
>> Makes sense?
>>
>>
>> On 31 July 2014 09:39, Sebastien Blanc <scm.blanc at gmail.com> wrote:
>>
>>> Well, several VariantMetricInformation instances can have the same
>>> VariantID, at each send , one is created :
>>> https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/push%2Fsrc%2Fmain%2Fjava%2Forg%2Fjboss%2Faerogear%2Funifiedpush%2Fmessage%2FSenderServiceImpl.java#L133
>>>
>>>
>>>
>>> On Thu, Jul 31, 2014 at 9:34 AM, Daniel Bevenius <
>>> daniel.bevenius at gmail.com> wrote:
>>>
>>>> Is this because variantFour and variantFive have the same variantId
>>>> (231543432434)? When added to the map only one will exist later
>>>> in findTopThreeBusyVariantIDs.
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> On 31 July 2014 09:20, Sebastien Blanc <scm.blanc at gmail.com> wrote:
>>>>
>>>>> Morning Peeps,
>>>>>
>>>>> I'm currently trying to fix AGPUSH-848[1].
>>>>> Basically, the number of receivers shown in the top3 list is not
>>>>> always accurate.
>>>>>
>>>>> I suspect that something is wrong with this query :
>>>>>
>>>>>
>>>>> https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/model/jpa/src/main/java/org/jboss/aerogear/unifiedpush/jpa/dao/impl/JPAPushMessageInformationDao.java#L99-L104
>>>>>
>>>>> I have change this test case :
>>>>>
>>>>> https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/model%2Fjpa%2Fsrc%2Ftest%2Fjava%2Forg%2Fjboss%2Faerogear%2Funifiedpush%2Fjpa%2FPushMessageInformationDaoTest.java#L251
>>>>>
>>>>> By adding just one VariantInformation[2] and now the test is failing
>>>>> and I have no idea why, so I would aprreciate a second eye on this.
>>>>>
>>>>> I'm probably missing something obvious but I can not see it right now
>>>>> :)
>>>>>
>>>>> Sebi
>>>>>
>>>>>
>>>>> [1]https://issues.jboss.org/browse/AGPUSH-848
>>>>> [2]
>>>>> https://gist.github.com/sebastienblanc/ea34e7f9fdafbc0785f2#file-gistfile1-java-L30-L35
>>>>>
>>>>> _______________________________________________
>>>>> aerogear-dev mailing list
>>>>> aerogear-dev at lists.jboss.org
>>>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> aerogear-dev mailing list
>>>> aerogear-dev at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>>>
>>>
>>>
>>> _______________________________________________
>>> aerogear-dev mailing list
>>> aerogear-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>>
>>
>>
>> _______________________________________________
>> aerogear-dev mailing list
>> aerogear-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>
>
>
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20140731/200ac01d/attachment-0001.html