>Hi, all
>I think so, I will start investigating the performance gap between the two
>>and send a performance report. Then I will sort out the map we need to
>replace.
At 2018-12-13 10:22:20, "yuhang xiu" <[email protected]> wrote:
>Hi, all
>Recently, @y1317427691 claimed this part of the work on github, he will
>start in the near future.
>According to my personal assumptions, we need to investigate the
>performance of longobjectmap first, and then start to sort out the map we
>need to replace.
>What do you think?
>
>
>Huxing Zhang <[email protected]> 于2018年12月7日周五 上午11:15写道:
>
>> On Tue, Dec 4, 2018 at 9:43 PM Huxing Zhang <[email protected]> wrote:
>> >
>> > Hi,
>> >
>> > On Tue, Dec 4, 2018 at 4:24 PM yuhang xiu <[email protected]> wrote:
>> > >
>> > > Hi, community
>> > > At present, there are many Map<Long, Object> type maps in dubbo. When
>> using
>> > > this kind of map, jdk performs automatic package.
>> > >
>> > > We can use longObjectMap(or intObjectMap or sth else) to replace these
>> maps
>> > > and add some performance.
>> >
>> > Do you have some benchmark result for this?
>> >
>> > >
>> > > Similarly, I have opened an issue[1] to track the progress. Of course,
>> I
>> > > will still find a new person to complete the work, we will comb and
>> replace
>> > > these maps together. I have found a person. We will start the
>> organizing
>> > > work and the program, and will be synchronized to the mailing list and
>> > > issue. (
>> > > He still has some work at the moment, so the discussion may be delayed
>> for
>> > > a few days)
>> >
>> > Can he be on the mailing list to discuss this?
>> >
>> > >
>> > > By the way, there are a few pr content that is simple, I think we can
>> do a
>> > > review and merge them. The issue list is here:
>> > > https://github.com/apache/incubator-dubbo/pull/2831
>> > > https://github.com/apache/incubator-dubbo/pull/2809
>>
>> I have finished the following two prs:
>>
>> https://github.com/apache/incubator-dubbo/pull/2802
>> https://github.com/apache/incubator-dubbo/pull/2788
>>
>>
>>
>> > >
>> > > Can someone take a look?
>> >
>> > I saw your comments on some of the pull request.
>> > I can take some of them this week.
>> >
>> > >
>> > > [1]https://github.com/apache/incubator-dubbo/issues/2882
>> >
>> >
>> >
>> > --
>> > Best Regards!
>> > Huxing
>>
>>
>>
>> --
>> Best Regards!
>> Huxing
>>