We had a long discussion about this in the design doc:
https://docs.google.com/document/d/1XfgF4jDXZDVIEWQPx6Y4glgeTTswAAxw6j8dPDAtoeI/edit?disco=AAAAAr0igSQ

TL:DR; is that we kept GET_WEIGHTS because we had its counter part
UPDATE_WEIGHTS.

On Thu, Jun 23, 2016 at 8:16 PM, haosdent <haosd...@gmail.com> wrote:

> Hi, @adam @vinodkone @yongqiao @zhouxin When we document the GET_ROLES and
> GET_WEIGHTS Operator APIs in this patch
> https://reviews.apache.org/r/49157/#comment204491 , we notice that
> there are duplicated information between GET_ROLES and GET_WEIGHTS. Both of
> them can return the roles and their weights expect GET_ROLES returns more
> information and GET_WEIGHTS has more exactly permissions check.
>
> So seems  GET_WEIGHTS is not necessary because we could get the
> corresponding information from GET_ROLES, do you think we should
> remove GET_WEIGHTS in Operator APIs?
>
> --
> Best Regards,
> Haosdent Huang
>

Reply via email to