Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id D910B200D10 for ; Sun, 24 Sep 2017 13:55:06 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id D77241609E6; Sun, 24 Sep 2017 11:55:06 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 286861609D7 for ; Sun, 24 Sep 2017 13:55:06 +0200 (CEST) Received: (qmail 74207 invoked by uid 500); 24 Sep 2017 11:55:05 -0000 Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list issues@hbase.apache.org Received: (qmail 74195 invoked by uid 99); 24 Sep 2017 11:55:05 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 24 Sep 2017 11:55:05 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id D6D0CD9DC0 for ; Sun, 24 Sep 2017 11:55:04 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -99.202 X-Spam-Level: X-Spam-Status: No, score=-99.202 tagged_above=-999 required=6.31 tests=[KAM_ASCII_DIVIDERS=0.8, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, USER_IN_WHITELIST=-100] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id c1gJQkey7egH for ; Sun, 24 Sep 2017 11:55:04 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTP id 7E58F5FCDA for ; Sun, 24 Sep 2017 11:55:03 +0000 (UTC) Received: from jira-lw-us.apache.org (unknown [207.244.88.139]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id 8C239E00DF for ; Sun, 24 Sep 2017 11:55:02 +0000 (UTC) Received: from jira-lw-us.apache.org (localhost [127.0.0.1]) by jira-lw-us.apache.org (ASF Mail Server at jira-lw-us.apache.org) with ESMTP id D3AF124215 for ; Sun, 24 Sep 2017 11:55:00 +0000 (UTC) Date: Sun, 24 Sep 2017 11:55:00 +0000 (UTC) From: "Anoop Sam John (JIRA)" To: issues@hbase.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (HBASE-18807) Remove PB references from Observers for Quotas MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 archived-at: Sun, 24 Sep 2017 11:55:07 -0000 [ https://issues.apache.org/jira/browse/HBASE-18807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16178171#comment-16178171 ] Anoop Sam John commented on HBASE-18807: ---------------------------------------- Sorry for being late here. GlobalQuotaSettings , I can see states like QuotaProtos.Throttle and getters for that. So we expect CP users to make use of these PB objects via the getters and do some thing? This also should NOT be ideally happening.. PB is our internal stuff and we should not expose that to the users. If we are not doing that, we can change our PB stuff (Like the shading what we did recently) with out much worry. > Remove PB references from Observers for Quotas > ---------------------------------------------- > > Key: HBASE-18807 > URL: https://issues.apache.org/jira/browse/HBASE-18807 > Project: HBase > Issue Type: Sub-task > Reporter: Josh Elser > Assignee: Josh Elser > Fix For: 3.0.0, 2.0.0-alpha-4 > > Attachments: HBASE-18807.001.branch-2.patch, HBASE-18807.002.branch-2.patch, HBASE-18807.003.branch-2.patch, HBASE-18807.004.branch-2.patch > > > Break-out from the parent: > Same idea, just applied to the Observer methods for pre/post quota operations. Requires changes to MasterQuotaManager and the QuotaSettings implementations as some business logic is written on the PB objects directly. -- This message was sent by Atlassian JIRA (v6.4.14#64029)