[GitHub] PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance

2017-11-23 Thread GitBox
PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance URL: https://github.com/apache/cloudstack/pull/2260#issuecomment-346599411 @rhtyd , I was actually waiting for @marcaurele's reply about a Marvin test instead of unit test.

[GitHub] PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance

2017-10-31 Thread GitBox
PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance URL: https://github.com/apache/cloudstack/pull/2260#issuecomment-340772630 @marcaurele, since this change is related to database queries, a Marvin test will help because

[GitHub] PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance

2017-10-09 Thread git
PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance URL: https://github.com/apache/cloudstack/pull/2260#issuecomment-335360041 @rhtyd, I am writing Marvin test case for this code change to test the listTemplate API, instead

[GitHub] PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance

2017-09-27 Thread git
PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance URL: https://github.com/apache/cloudstack/pull/2260#issuecomment-332729267 @rhtyd : Could you please merge this change ? There are 2 Test LGTM and a Code LGTM, and no

[GitHub] PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance

2017-09-27 Thread git
PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance URL: https://github.com/apache/cloudstack/pull/2260#issuecomment-332544636 tag:mergeready This is an