You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Josh Elser (JIRA)" <ji...@apache.org> on 2018/07/24 16:03:00 UTC
[jira] [Commented] (HBASE-20929)
IncreasingToUpperBoundRegionSplitPolicy -> shouldSplit -> Log.debug should
be enclosed inside LOG.isdebugEnabled()
[ https://issues.apache.org/jira/browse/HBASE-20929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16554444#comment-16554444 ]
Josh Elser commented on HBASE-20929:
------------------------------------
[~vishk], would be better to switch this over to SLF4j API and use the {{{}}} replacement markers to avoid the extra if-clause.
You planning to work on this?
> IncreasingToUpperBoundRegionSplitPolicy -> shouldSplit -> Log.debug should be enclosed inside LOG.isdebugEnabled()
> ------------------------------------------------------------------------------------------------------------------
>
> Key: HBASE-20929
> URL: https://issues.apache.org/jira/browse/HBASE-20929
> Project: HBase
> Issue Type: Bug
> Reporter: Vishal Khandelwal
> Assignee: Vishal Khandelwal
> Priority: Minor
>
> Class IncreasingToUpperBoundRegionSplitPolicy.
>
> if (size > sizeToCheck) {
> LOG.debug("ShouldSplit because " + store.getColumnFamilyName() + " size=" + size
> + ", sizeToCheck=" + sizeToCheck + ", regionsWithCommonTable="
> + tableRegionsCount);
> foundABigStore = true;
> }
>
> LOG.debug("Failed getOnlineRegions " + tablename, e);
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)