[openstack-dev] [ceilometer] raise clientSideError or do for user

2014-04-14 Thread ZhiQiang Fan
Hi, developers, For fixing bug https://launchpad.net/bugs/1304886, I uploaded a patch https://review.openstack.org/#/c/86501/, when review this patch, there are two different kinds of opinions, I don't know which is the better choice, so I ask for help here. The patch aims at disallow user

Re: [openstack-dev] [ceilometer] raise clientSideError or do for user

2014-04-14 Thread Julien Danjou
On Mon, Apr 14 2014, ZhiQiang Fan wrote: Hi, developers, For fixing bug https://launchpad.net/bugs/1304886, I uploaded a patch https://review.openstack.org/#/c/86501/, when review this patch, there are two different kinds of opinions, I don't know which is the better choice, so I ask for

Re: [openstack-dev] [ceilometer] raise clientSideError or do for user

2014-04-14 Thread ZhiQiang Fan
thanks for the quick reply :) On Mon, Apr 14, 2014 at 9:23 PM, Julien Danjou jul...@danjou.info wrote: On Mon, Apr 14 2014, ZhiQiang Fan wrote: Hi, developers, For fixing bug https://launchpad.net/bugs/1304886, I uploaded a patch https://review.openstack.org/#/c/86501/, when review