Re: [Carbon-dev] Updated Invitation: [Code Review] [Group B] Message box core - connection reuse @ Tue Oct 4 10:30am - 11:30am (nu...@wso2.com)
Hi all, Code review notes: 1. Current implementation of SQS(message-box) does not use JMS connection reuse. Instead, It creates queue connections(tcp connections) per message. The reason behind this is to handle visibility timeout using JMS transaction CLIENT_ACKNOWLEDGE mode. When a message is received, the queue connection is created and session created with CLIENT_ACKNOWLEDGE enabled. If the user does not delete the received message, the session is closed without acknowledge. As a result the received message will be available or visible to other users to receive again. If the user delete the message after receiving it, message is acknowledged. If we acknowledge a message in the middle of the queue, the messages up-to that message also acknowledged. This causes to use per connection per message. Action item: There was an implementation to cache connection, and it was removed due to not working properly with multiple receivers. This should be fixed and tested. And look for other alternative to implement above use case. 2. The behavior of some JMS API calls is different with different providers, ex. create persistent queues, queue receiver receive methods. So add comments on such methods used so that it can be identified fast. 3. Add error logs before throwing exceptions. After two weeks, these improvements will be reviewed again. If I have missed anything, please add comments. Thank you. On Tue, Oct 4, 2011 at 10:27 AM, Nuwan Bandara nu...@wso2.com wrote: GROUP B, Reminder !!! On Tue, Oct 4, 2011 at 10:02 AM, Shammi Jayasinghe sha...@wso2.comwrote: This event has been changed. more details »https://www.google.com/calendar/event?action=VIEWeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en [Code Review] [Group B] Message box core - connection reuse *Changed: *crcuible link : https://wso2.org/crucible/cru/COMPONENT-63http://www.google.com/url?q=https%3A%2F%2Fwso2.org%2Fcrucible%2Fcru%2FCOMPONENT-63ust=131770992217usg=AFQjCNFMgu4Hh2Xo9i91MU9Kcyq0HT7Oew *When* Tue Oct 4 10:30am – 11:30am Colombo *Where* #59 - Board Room (maphttp://maps.google.lk/maps?q=%2359+-+Board+Roomhl=en ) *Calendar* nu...@wso2.com *Who* • sha...@wso2.com - organizer • Chamara Ariyarathne • Achala Aponso • Deependra Ariyadewa • Dimuthu Leelarathne • Manjula Rathnayake • Nuwan Bandara • carbon-dev@wso2.org • Krishantha Samaraweera • Nuwan Wimalasekara • Sanjeewa Malalgoda • Anjana Fernando • Nandika Jayawardana • Ajith Vitharana • Thilini Ishaka • Jasintha Dasanayake Going? ***Yeshttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=1tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en- Maybehttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=3tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en- Nohttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=2tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en ***more options »https://www.google.com/calendar/event?action=VIEWeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en Invitation from Google Calendar https://www.google.com/calendar/ You are receiving this email at the account nu...@wso2.com because you are subscribed for updated invitations on calendar nu...@wso2.com. To stop receiving these notifications, please log in to https://www.google.com/calendar/ and change your notification settings for this calendar. -- *Thanks Regards, Nuwan Bandara Senior Software Engineer WSO2 Inc. | http://wso2.com lean . enterprise . middleware http://nuwan.bandara.co * http://www.nuwanbando.com/ -- Manjula Rathnayaka Software Engineer WSO2, Inc. Mobile:+94 77 743 1987 ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
Re: [Carbon-dev] Updated Invitation: [Code Review] [Group B] Message box core - connection reuse @ Tue Oct 4 10:30am - 11:30am (nu...@wso2.com)
Where are the notes? On Tue, Oct 4, 2011 at 10:27 AM, Nuwan Bandara nu...@wso2.com wrote: GROUP B, Reminder !!! On Tue, Oct 4, 2011 at 10:02 AM, Shammi Jayasinghe sha...@wso2.comwrote: This event has been changed. more details »https://www.google.com/calendar/event?action=VIEWeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en [Code Review] [Group B] Message box core - connection reuse *Changed: *crcuible link : https://wso2.org/crucible/cru/COMPONENT-63http://www.google.com/url?q=https%3A%2F%2Fwso2.org%2Fcrucible%2Fcru%2FCOMPONENT-63ust=131770992217usg=AFQjCNFMgu4Hh2Xo9i91MU9Kcyq0HT7Oew *When* Tue Oct 4 10:30am – 11:30am Colombo *Where* #59 - Board Room (maphttp://maps.google.lk/maps?q=%2359+-+Board+Roomhl=en ) *Calendar* nu...@wso2.com *Who* • sha...@wso2.com - organizer • Chamara Ariyarathne • Achala Aponso • Deependra Ariyadewa • Dimuthu Leelarathne • Manjula Rathnayake • Nuwan Bandara • carbon-dev@wso2.org • Krishantha Samaraweera • Nuwan Wimalasekara • Sanjeewa Malalgoda • Anjana Fernando • Nandika Jayawardana • Ajith Vitharana • Thilini Ishaka • Jasintha Dasanayake Going? ***Yeshttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=1tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en- Maybehttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=3tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en- Nohttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=2tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en ***more options »https://www.google.com/calendar/event?action=VIEWeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en Invitation from Google Calendar https://www.google.com/calendar/ You are receiving this email at the account nu...@wso2.com because you are subscribed for updated invitations on calendar nu...@wso2.com. To stop receiving these notifications, please log in to https://www.google.com/calendar/ and change your notification settings for this calendar. -- *Thanks Regards, Nuwan Bandara Senior Software Engineer WSO2 Inc. | http://wso2.com lean . enterprise . middleware http://nuwan.bandara.co * http://www.nuwanbando.com/ ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev -- *Afkham Azeez* Director of Architecture; WSO2, Inc.; http://wso2.com Member; Apache Software Foundation; http://www.apache.org/ * http://www.apache.org/** email: **az...@wso2.com* az...@wso2.com* cell: +94 77 3320919 blog: **http://blog.afkham.org* http://blog.afkham.org* twitter: **http://twitter.com/afkham_azeez*http://twitter.com/afkham_azeez * linked-in: **http://lk.linkedin.com/in/afkhamazeez* * * *Lean . Enterprise . Middleware* ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
Re: [Carbon-dev] Updated Invitation: [Code Review] [Group B] Message box core - connection reuse @ Tue Oct 4 10:30am - 11:30am (nu...@wso2.com)
On Tue, Oct 4, 2011 at 2:41 PM, Manjula Rathnayake manju...@wso2.comwrote: Hi all, Code review notes: 1. Current implementation of SQS(message-box) does not use JMS connection reuse. Instead, It creates queue connections(tcp connections) per message. The reason behind this is to handle visibility timeout using JMS transaction CLIENT_ACKNOWLEDGE mode. When a message is received, the queue connection is created and session created with CLIENT_ACKNOWLEDGE enabled. If the user does not delete the received message, the session is closed without acknowledge. As a result the received message will be available or visible to other users to receive again. If the user delete the message after receiving it, message is acknowledged. If we acknowledge a message in the middle of the queue, the messages up-to that message also acknowledged. This causes to use per connection per message. As per offline chat with amila and going through the previous connection cache code, We can implement the connection re-use. This is just a correction to above statement. We can have connection cache enabled per user and one session per message retrieval. And it is not one connection per message, however In the current implementation, it is one connection per message. We will implement this and update on this. Action item: There was an implementation to cache connection, and it was removed due to not working properly with multiple receivers. This should be fixed and tested. And look for other alternative to implement above use case. 2. The behavior of some JMS API calls is different with different providers, ex. create persistent queues, queue receiver receive methods. So add comments on such methods used so that it can be identified fast. 3. Add error logs before throwing exceptions. After two weeks, these improvements will be reviewed again. If I have missed anything, please add comments. Thank you. On Tue, Oct 4, 2011 at 10:27 AM, Nuwan Bandara nu...@wso2.com wrote: GROUP B, Reminder !!! On Tue, Oct 4, 2011 at 10:02 AM, Shammi Jayasinghe sha...@wso2.comwrote: This event has been changed. more details »https://www.google.com/calendar/event?action=VIEWeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en [Code Review] [Group B] Message box core - connection reuse *Changed: *crcuible link : https://wso2.org/crucible/cru/COMPONENT-63http://www.google.com/url?q=https%3A%2F%2Fwso2.org%2Fcrucible%2Fcru%2FCOMPONENT-63ust=131770992217usg=AFQjCNFMgu4Hh2Xo9i91MU9Kcyq0HT7Oew *When* Tue Oct 4 10:30am – 11:30am Colombo *Where* #59 - Board Room (maphttp://maps.google.lk/maps?q=%2359+-+Board+Roomhl=en ) *Calendar* nu...@wso2.com *Who* • sha...@wso2.com - organizer • Chamara Ariyarathne • Achala Aponso • Deependra Ariyadewa • Dimuthu Leelarathne • Manjula Rathnayake • Nuwan Bandara • carbon-dev@wso2.org • Krishantha Samaraweera • Nuwan Wimalasekara • Sanjeewa Malalgoda • Anjana Fernando • Nandika Jayawardana • Ajith Vitharana • Thilini Ishaka • Jasintha Dasanayake Going? ***Yeshttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=1tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en- Maybehttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=3tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en- Nohttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=2tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en ***more options »https://www.google.com/calendar/event?action=VIEWeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en Invitation from Google Calendar https://www.google.com/calendar/ You are receiving this email at the account nu...@wso2.com because you are subscribed for updated invitations on calendar nu...@wso2.com. To stop receiving these notifications, please log in to https://www.google.com/calendar/ and change your notification settings for this calendar. -- *Thanks Regards, Nuwan Bandara Senior Software Engineer WSO2 Inc. | http://wso2.com lean . enterprise . middleware http://nuwan.bandara.co * http://www.nuwanbando.com/ -- Manjula Rathnayaka Software Engineer WSO2, Inc. Mobile:+94 77 743 1987 -- Manjula Rathnayaka Software Engineer WSO2, Inc. Mobile:+94 77 743 1987 ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
Re: [Carbon-dev] Updated Invitation: [Code Review] [Group B] Message box core - connection reuse @ Tue Oct 4 10:30am - 11:30am (nu...@wso2.com)
GROUP B, Reminder !!! On Tue, Oct 4, 2011 at 10:02 AM, Shammi Jayasinghe sha...@wso2.com wrote: This event has been changed. more details »https://www.google.com/calendar/event?action=VIEWeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en [Code Review] [Group B] Message box core - connection reuse *Changed: *crcuible link : https://wso2.org/crucible/cru/COMPONENT-63http://www.google.com/url?q=https%3A%2F%2Fwso2.org%2Fcrucible%2Fcru%2FCOMPONENT-63ust=131770992217usg=AFQjCNFMgu4Hh2Xo9i91MU9Kcyq0HT7Oew *When* Tue Oct 4 10:30am – 11:30am Colombo *Where* #59 - Board Room (maphttp://maps.google.lk/maps?q=%2359+-+Board+Roomhl=en ) *Calendar* nu...@wso2.com *Who* • sha...@wso2.com - organizer • Chamara Ariyarathne • Achala Aponso • Deependra Ariyadewa • Dimuthu Leelarathne • Manjula Rathnayake • Nuwan Bandara • carbon-dev@wso2.org • Krishantha Samaraweera • Nuwan Wimalasekara • Sanjeewa Malalgoda • Anjana Fernando • Nandika Jayawardana • Ajith Vitharana • Thilini Ishaka • Jasintha Dasanayake Going? ***Yeshttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=1tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en- Maybehttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=3tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en- Nohttps://www.google.com/calendar/event?action=RESPONDeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20rst=2tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en ***more options »https://www.google.com/calendar/event?action=VIEWeid=MThraHVoMG5icGNyMDNuMHRwNnU3YzRuMjAgbnV3YW5Ad3NvMi5jb20tok=MTUjc2hhbW1pQHdzbzIuY29tOWI0MzU0MjhhNjBiY2IwMDg2NDE4Y2JiY2FkODQ2ZmRmZjQwMWM1MActz=Asia%2FColombohl=en Invitation from Google Calendar https://www.google.com/calendar/ You are receiving this email at the account nu...@wso2.com because you are subscribed for updated invitations on calendar nu...@wso2.com. To stop receiving these notifications, please log in to https://www.google.com/calendar/ and change your notification settings for this calendar. -- *Thanks Regards, Nuwan Bandara Senior Software Engineer WSO2 Inc. | http://wso2.com lean . enterprise . middleware http://nuwan.bandara.co * http://www.nuwanbando.com/ ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev