Return-Path: Delivered-To: apmail-hbase-dev-archive@www.apache.org Received: (qmail 77487 invoked from network); 15 Dec 2010 17:39:55 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 15 Dec 2010 17:39:55 -0000 Received: (qmail 1015 invoked by uid 500); 15 Dec 2010 17:39:54 -0000 Delivered-To: apmail-hbase-dev-archive@hbase.apache.org Received: (qmail 935 invoked by uid 500); 15 Dec 2010 17:39:54 -0000 Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hbase.apache.org Delivered-To: mailing list dev@hbase.apache.org Received: (qmail 918 invoked by uid 99); 15 Dec 2010 17:39:53 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 15 Dec 2010 17:39:53 +0000 X-ASF-Spam-Status: No, hits=2.2 required=10.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (athena.apache.org: local policy) Received: from [209.85.216.176] (HELO mail-qy0-f176.google.com) (209.85.216.176) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 15 Dec 2010 17:39:47 +0000 Received: by qyk10 with SMTP id 10so2161812qyk.14 for ; Wed, 15 Dec 2010 09:39:26 -0800 (PST) Received: by 10.224.36.208 with SMTP id u16mr6470055qad.299.1292434766376; Wed, 15 Dec 2010 09:39:26 -0800 (PST) MIME-Version: 1.0 Received: by 10.220.190.75 with HTTP; Wed, 15 Dec 2010 09:39:05 -0800 (PST) X-Originating-IP: [64.105.168.204] In-Reply-To: <20101215023655.24974.19248@ip-10-202-7-187.ec2.internal> References: <20101215021622.3648.4186@ip-10-202-7-187.ec2.internal> <20101215023655.24974.19248@ip-10-202-7-187.ec2.internal> From: Ted Dunning Date: Wed, 15 Dec 2010 09:39:05 -0800 Message-ID: Subject: Re: Review Request: Allow Observers to completely override base function To: dev@hbase.apache.org Cc: Jonathan Gray , Mingjie Lai , Andrew Purtell , jiraposter@review.hbase.org, Ryan Rawson Content-Type: multipart/alternative; boundary=0015175cb0ba2530f50497766ba8 --0015175cb0ba2530f50497766ba8 Content-Type: text/plain; charset=ISO-8859-1 Indeed, if you look at the implementation of AtomicInteger, it is mostly just a volatile int. On Tue, Dec 14, 2010 at 6:36 PM, Ryan Rawson wrote: > src/main/java/org/apache/hadoop/hbase/regionserver/CoprocessorHost.java > > > unless you need CAS semantics, you can just use volatile here. We are > over-using the Atomic* stuff sometimes. --0015175cb0ba2530f50497766ba8--