You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by "Chia-Ping Tsai (JIRA)" <ji...@apache.org> on 2017/12/16 11:03:00 UTC

[jira] [Created] (HBASE-19531) Remove needless volatile declaration

Chia-Ping Tsai created HBASE-19531:
--------------------------------------

             Summary: Remove needless volatile declaration
                 Key: HBASE-19531
                 URL: https://issues.apache.org/jira/browse/HBASE-19531
             Project: HBase
          Issue Type: Improvement
            Reporter: Chia-Ping Tsai
            Priority: Trivial
             Fix For: 2.0.0


{code:title=ZKNamespaceManager.java}
public class ZKNamespaceManager extends ZKListener {
  private static final Log LOG = LogFactory.getLog(ZKNamespaceManager.class);
  private final String nsZNode;
  private volatile NavigableMap<String,NamespaceDescriptor> cache; //here
{code}

{code:title=HBaseAdmin.java}
public class HBaseAdmin implements Admin {
  private static final Log LOG = LogFactory.getLog(HBaseAdmin.class);

  private ClusterConnection connection;

  private volatile Configuration conf; //here
{code}

{code:title=BufferedMutatorImpl.java}
public class BufferedMutatorImpl implements BufferedMutator {

  private static final Log LOG = LogFactory.getLog(BufferedMutatorImpl.class);

  private final ExceptionListener listener;

  private final TableName tableName;

  private final Configuration conf;
  private final ConcurrentLinkedQueue<Mutation> writeAsyncBuffer = new ConcurrentLinkedQueue<>();
  private final AtomicLong currentWriteBufferSize = new AtomicLong(0);
  /**
   * Count the size of {@link BufferedMutatorImpl#writeAsyncBuffer}.
   * The {@link ConcurrentLinkedQueue#size()} is NOT a constant-time operation.
   */
  private final AtomicInteger undealtMutationCount = new AtomicInteger(0);
  private volatile long writeBufferSize;  //here
{code}




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)