hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Solomon Duskis (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-12490) Replace uses of setAutoFlush(boolean, boolean)
Date Mon, 01 Dec 2014 18:43:12 GMT

    [ https://issues.apache.org/jira/browse/HBASE-12490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14230217#comment-14230217
] 

Solomon Duskis commented on HBASE-12490:
----------------------------------------

There's only one scenario where behavior might change, and that should hopefully be an obscure
case.  That obscure case does not affect the code in this commit directly AFAIK.

The changes i made from setAutoFlush(false, true) to setAutoFlushTo(false) assume that clearBufferOnFail
was previously true, which will call setAutoFlush(false, clearBufferOnFail) -> setAutoFlush(false,
true) .  If someone calls setAutoFlush(false) or setAutoFlush(false, false), and setAutoFlushTo(false)
is called, then clearBufferOnFail will remain false.

> Replace uses of setAutoFlush(boolean, boolean)
> ----------------------------------------------
>
>                 Key: HBASE-12490
>                 URL: https://issues.apache.org/jira/browse/HBASE-12490
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 2.0.0, 0.99.2
>            Reporter: Solomon Duskis
>            Assignee: Solomon Duskis
>         Attachments: HBASE-12490.patch, HBASE-12490B.patch, HBASE-12490B.patch, HBASE-12490B.patch,
HBASE-12490C.patch
>
>
> The various uses of setAutoFlush() seem to need some tlc.  There's a note in HTableInterface:
"@deprecated in 0.99 since setting clearBufferOnFail is deprecated. Use setAutoFlushTo(boolean)
instead."  It would be ideal to change all internal uses of setAutoFlush(boolean, boolean)
to use setAutoFlushTo, if possible.
> HTable.setAutoFlush(boolean, boolean) is used in a handful of places.  setAutoFlush(false,
false) has the same results as HTable.setAutoFlush(false).  Calling HTable.setAutoFlush(false,
true) has the same affect as Table.setAutoFlushTo(false), assuming HTable.setAutoFlush(false)
was not called previously (by default, the second parameter, clearBufferOnFail, is true and
should remain true according to the comments). 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message