You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2016/03/20 15:22:33 UTC
[jira] [Commented] (HBASE-15490) Remove duplicated
CompactionThroughputControllerFactory in branch-1
[ https://issues.apache.org/jira/browse/HBASE-15490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15203309#comment-15203309 ]
Hudson commented on HBASE-15490:
--------------------------------
SUCCESS: Integrated in HBase-1.4 #37 (See [https://builds.apache.org/job/HBase-1.4/37/])
HBASE-15490 Remove duplicated CompactionThroughputControllerFactory in (liyu: rev 17815ded74d3789b7474969cd105e8bd31e8bbaa)
* hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/CompactionThroughputControllerFactory.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/compactions/CompactionThroughputControllerFactory.java
> Remove duplicated CompactionThroughputControllerFactory in branch-1
> -------------------------------------------------------------------
>
> Key: HBASE-15490
> URL: https://issues.apache.org/jira/browse/HBASE-15490
> Project: HBase
> Issue Type: Bug
> Affects Versions: 1.3.0
> Reporter: Yu Li
> Assignee: Yu Li
> Fix For: 1.3.0
>
> Attachments: HBASE-15490.branch-1.patch, HBASE-15490.patch
>
>
> Currently there're two {{CompactionThroughputControllerFactory}} in our branch-1 code base (one in {{o.a.h.h.regionserver.compactions}} package, the other in {{o.a.h.h.regionserver.throttle}}) and both are in use.
> This is a regression of HBASE-14969 and only exists in branch-1. We should remove the one in {{o.a.h.h.regionserver.compactions}}, and change the default compaction throughput controller back to {{NoLimitThroughputController}} to keep compatible with previous branch-1 version
> Thanks [~ghelmling] for pointing out the issue.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)