Hi all,

Please find the code review feedback.


   - Make sure the setMappingTextList() logic checks for cases of
   '}}<attribname>{{' kind of scenarios
   - Rename setMappingTextList() to something more appropriate. Refactor
   methods to separate functionality properly.
   - Change the 'catch (Exception e)' to 'catch (Throwable e)' at the start
   of the thread for Kafka Output Adapter.

Thanks,
Lasantha

On 4 August 2015 at 11:46, Ramindu De Silva <[email protected]> wrote:

> *Ramindu De Silva has accepted this invitation.*
> [Code Review] Performance Improvements to XML/JSON output mapping
> *When*
> Tue 4 Aug 2015 14:00 – 15:00 Colombo
> *Where*
> LK Trace Mezzanine Floor Meeting Room - Gondor Ext:1036212 (map
> <https://maps.google.com/maps?q=LK+Trace+Mezzanine+Floor+Meeting+Room+-+Gondor+Ext:1036212&hl=en-GB>
> )
> *Video call*
> https://plus.google.com/hangouts/_/wso2.com/lasantha
> <https://plus.google.com/hangouts/_/wso2.com/lasantha?hceid=bGFzYW50aGFAd3NvMi5jb20.mga53m844ntcgfmpfgi150pb10>
> *Calendar*
> [email protected]
> *Who*
> •
> Lasantha Fernando- organiser
> •
> Ramindu De Silva
> •
> Damith Wickramasinghe
> •
> [email protected]
> •
> [email protected]
> •
> Mohanadarshan Vivekanandalingam
> •
> Sriskandarajah Suhothayan
> •
> [email protected]
> •
> Tishan Dahanayakage
> •
> Sajith Ravindra
> •
> Tharik Kanaka
> •
> Ayashkantha Ramasinghe
> •
> [email protected]
> •
> Dilini Muthumala
>
> Invitation from Google Calendar <https://www.google.com/calendar/>
>
> You are receiving this email at the account [email protected] because you
> are subscribed for invitation replies on calendar [email protected].
>
> To stop receiving these emails, please log in to
> https://www.google.com/calendar/ and change your notification settings
> for this calendar.
>
> Forwarding this invitation could allow any recipient to modify your RSVP
> response. Learn More
> <https://support.google.com/calendar/answer/37135#forwarding>.
>



-- 
*Lasantha Fernando*
Senior Software Engineer - Data Technologies Team
WSO2 Inc. http://wso2.com

email: [email protected]
mobile: (+94) 71 5247551
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to