Also, * Renaming the "insert" operation in Analytics Record Store / Data Service to "put", since this operation does an 'insert or update' operation.
Cheers, Anjana. On Tue, Feb 24, 2015 at 10:28 PM, Anjana Fernando <[email protected]> wrote: > Code Review Notes:- > > * Create sub-packages in analytics data source module to separate > analytics record store / file system related interfaces/classes > * Put file name when logging an FS related exception > * Missing null checks when closing streams. > > Cheers, > Anjana. > > On Tue, Feb 24, 2015 at 9:16 AM, [email protected] <[email protected]> wrote: > >> This event has been changed. >> more details » >> <https://www.google.com/calendar/event?action=VIEW&eid=aDlzb2d2bHBkbGk5aXBoZnNkbXZqbmN2am8gZGV2QHdzbzIub3Jn&tok=MTUjYW5qYW5hQHdzbzIuY29tMDhjZjdhZTljNzk1ZTBlNDZhZThhNzkzY2Q3MWM5OWMxMmZmMDJlOQ&ctz=Asia/Colombo&hl=en> >> BAM DAL Code Review >> Code review for BAM 3.0 DAL implementation. >> https://redmine.wso2.com/issues/3520 >> <https://www.google.com/url?q=https%3A%2F%2Fredmine.wso2.com%2Fissues%2F3520&usd=2&usg=AFQjCNEYVI0MBCLgAMtrJXvpORwHpu9wGg> >> . >> *When* >> *Changed: *Tue Feb 24, 2015 3pm – 4pm Colombo >> *Where* >> LK Trace Mezzanine Floor Board Room - Shire (map >> <https://maps.google.lk/maps?q=LK+Trace+Mezzanine+Floor+Board+Room+-+Shire&hl=en> >> ) >> *Video call* >> https://plus.google.com/hangouts/_/wso2.com/bam-dal-code >> <https://plus.google.com/hangouts/_/wso2.com/bam-dal-code?hceid=YW5qYW5hQHdzbzIuY29t.h9sogvlpdli9iphfsdmvjncvjo> >> *Calendar* >> Anjana Fernando >> *Who* >> • >> Anjana Fernando - organizer >> • >> Niranda Perera >> • >> Sriskandarajah Suhothayan >> • >> Gimantha Bandara >> • >> Nirmal Fernando >> • >> Sinthuja Ragendran >> • >> Srinath Perera >> • >> Dunith Dhanushka >> • >> Malith Dhanushka >> • >> Pubudu Dodangoda >> • >> Mohammed Fawsan >> • >> Gihan Anuruddha >> • >> WSO2 Developers' List >> • >> Seshika Fernando >> >> Going? *Yes >> <https://www.google.com/calendar/event?action=RESPOND&eid=aDlzb2d2bHBkbGk5aXBoZnNkbXZqbmN2am8gZGV2QHdzbzIub3Jn&rst=1&tok=MTUjYW5qYW5hQHdzbzIuY29tMDhjZjdhZTljNzk1ZTBlNDZhZThhNzkzY2Q3MWM5OWMxMmZmMDJlOQ&ctz=Asia/Colombo&hl=en> >> - Maybe >> <https://www.google.com/calendar/event?action=RESPOND&eid=aDlzb2d2bHBkbGk5aXBoZnNkbXZqbmN2am8gZGV2QHdzbzIub3Jn&rst=3&tok=MTUjYW5qYW5hQHdzbzIuY29tMDhjZjdhZTljNzk1ZTBlNDZhZThhNzkzY2Q3MWM5OWMxMmZmMDJlOQ&ctz=Asia/Colombo&hl=en> >> - No >> <https://www.google.com/calendar/event?action=RESPOND&eid=aDlzb2d2bHBkbGk5aXBoZnNkbXZqbmN2am8gZGV2QHdzbzIub3Jn&rst=2&tok=MTUjYW5qYW5hQHdzbzIuY29tMDhjZjdhZTljNzk1ZTBlNDZhZThhNzkzY2Q3MWM5OWMxMmZmMDJlOQ&ctz=Asia/Colombo&hl=en>* >> more options » >> <https://www.google.com/calendar/event?action=VIEW&eid=aDlzb2d2bHBkbGk5aXBoZnNkbXZqbmN2am8gZGV2QHdzbzIub3Jn&tok=MTUjYW5qYW5hQHdzbzIuY29tMDhjZjdhZTljNzk1ZTBlNDZhZThhNzkzY2Q3MWM5OWMxMmZmMDJlOQ&ctz=Asia/Colombo&hl=en> >> >> Invitation from Google Calendar <https://www.google.com/calendar/> >> >> You are receiving this courtesy email at the account [email protected] >> because you are an attendee of this event. >> >> To stop receiving future updates for this event, decline this event. >> Alternatively you can sign up for a Google account at >> https://www.google.com/calendar/ and control your notification settings >> for your entire calendar. >> >> _______________________________________________ >> Dev mailing list >> [email protected] >> http://wso2.org/cgi-bin/mailman/listinfo/dev >> >> > > > -- > *Anjana Fernando* > Senior Technical Lead > WSO2 Inc. | http://wso2.com > lean . enterprise . middleware > -- *Anjana Fernando* Senior Technical Lead WSO2 Inc. | http://wso2.com lean . enterprise . middleware
_______________________________________________ Dev mailing list [email protected] http://wso2.org/cgi-bin/mailman/listinfo/dev
