[
https://issues.apache.org/jira/browse/HBASE-19531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16295349#comment-16295349
]
Hudson commented on HBASE-19531:
--------------------------------
FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #4249 (See
[https://builds.apache.org/job/HBase-Trunk_matrix/4249/])
HBASE-19531 Remove needless volatile declaration (chia7712: rev
9d0c7c6dfbcba0907cbbc2244eac570fcc4d58a5)
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/ZKNamespaceManager.java
* (edit)
hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
* (edit)
hbase-client/src/main/java/org/apache/hadoop/hbase/client/BufferedMutatorImpl.java
> 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
> Assignee: Yun-Chi Shih
> Priority: Trivial
> Labels: beginner
> Fix For: 2.0.0
>
> Attachments: HBASE-19531.v0.patch, HBASE-19531.v1.patch
>
>
> {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)
