Due to my own confusion and bad I found I found my PR is containing more commits and more file modification other than mine. So avoid the confusion I created new PR and close the old one. This PR containing the all the review comments from old PR. Please review mine this PR <https://github.com/apache/incubator-dubbo/pull/3090>
https://github.com/apache/incubator-dubbo/pull/3090 Sorry for the inconvenience caused. On Thu, Dec 27, 2018 at 3:24 PM yuhang xiu <[email protected]> wrote: > Hi, > > I simply checked your pr and I agree with you. I will review it. > Nice work! > > Imteyaz Khan <[email protected]> 于2018年12月27日周四 下午5:35写道： > > > Hi All, > > I have raised a separate PR for the same issue 3026. > > > > https://github.com/apache/incubator-dubbo/pull/3080 > > > > As my old PR was containing may commits of others and reviewer would like > > me to clear those commits and I tried hard to achieve the same but failed > > so landed up creating new PR > > <https://github.com/apache/incubator-dubbo/pull/3080>. > > > > Please note :: older PR > > <https://github.com/apache/incubator-dubbo/pull/3027> 3027 has been > closed > > but all the comments are applicable here. > > > > On Thu, Dec 20, 2018 at 5:35 PM Imteyaz Khan <[email protected]> > > wrote: > > > > > Hi All, > > > I have raised a PR < > > https://github.com/apache/incubator-dubbo/pull/3027>for > > > issue 3026 <https://github.com/apache/incubator-dubbo/issues/3026>. In > > > this PR , I am reducing the number of SimpleDateFormat object creation > > > from each log message to per thread wise and reusing it. Please review > > it. > > > > > > https://github.com/apache/incubator-dubbo/pull/3027 > > > > > >
