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

ChiaPing Tsai commented on HBASE-17488:
---------------------------------------

hi [~stack]
bq. What sort of benefit do you expect
prevent creating unused objects in the WALEdit's construction.

# If the cp#preBatchMutate returns true, the WALEdit is useless. So we should 
create the WALEdit after step 2.
# The cells came from cp should be counted because they are added into the 
WALEdit . The use case is the local index of phoenix 
# If the mutation contains the SKIP_WAL property, its cells aren't added into 
the WALEdit. So these cells shouldn't be counted.

> WALEdit should be lazily instantiated
> -------------------------------------
>
>                 Key: HBASE-17488
>                 URL: https://issues.apache.org/jira/browse/HBASE-17488
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: ChiaPing Tsai
>            Assignee: ChiaPing Tsai
>            Priority: Trivial
>             Fix For: 2.0.0, 1.4.0
>
>         Attachments: HBASE-17488.branch-1.v0.patch
>
>
> Some trivial improvement.
> # create the WALEdit on step 3 instead of step 2
> # count the cells from coprocessor
> # don’t count the mutations which contain the Durability.SKIP_WAL property



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

Reply via email to