Hi, keystone devs, Thank you for your many comments about my patch [1].
I modified this patch in reference to your comments. I want to get many opinions about this new Patch Set. Could you review it? [1]: https://review.openstack.org/#/c/261188/ Thank you, On Tue, 31 May 2016 20:46:36 +0900 koshiya maho <koshiya.m...@po.ntts.co.jp> wrote: > Hi, keystone devs, > > Thank you for your many opinions about request id mapping. > I fixed this patch [1] by the way that Brant and Cao gave me suggestions [2]. > Could you review it? > > [1] https://review.openstack.org/#/c/261188/ > [2] http://paste.openstack.org/show/495040/ > > Thank you, > > On Wed, 20 Apr 2016 16:37:31 -0700 > Morgan Fainberg <morgan.fainb...@gmail.com> wrote: > > > On Wed, Apr 13, 2016 at 6:07 AM, David Stanek <dsta...@dstanek.com> wrote: > > > > > On Wed, Apr 13, 2016 at 3:26 AM koshiya maho <koshiya.m...@po.ntts.co.jp> > > > wrote: > > > > > >> > > >> My request to all keystone cores to give their suggestions about the > > >> same. > > >> > > >> > > > I'll test this a little and see if I can see how it breaks. > > > > > > Overall I'm not really a fan of this design. It's just a hack to add > > > attributes where they don't belong. Long term I think this will be hard to > > > maintain. > > > > > > > > > > > If we want to return a response object we should return a response object. > > Returning a magic list with attributes (or a dict with attributes, etc) > > feels very, very wrong. > > > > I'm not going to block this design, but I wish we had something a bit > > better. > > > > --Morgan > > -- > Maho Koshiya > E-Mail : koshiya.m...@po.ntts.co.jp > > > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev -- Maho Koshiya E-Mail : koshiya.m...@po.ntts.co.jp __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev