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 >