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

James Taylor commented on PHOENIX-541:
--------------------------------------

Sorry to leave this hanging, [~gjacoby]. I think this looks good to go - 
definitely an improvement. I do think we should do these couple of other items 
(and you're right, it's only calculated/used in that one place), but we can do 
that in a separate JIRA:
- Make similar changes QueryServices.MAX_MUTATION_SIZE_ATTRIB - making it 
byte-based instead of row-count-based. Usage of this config parameter would be 
isolated to MutationState, I believe. We should be able to come up with an 
accurate size based on the underlying Mutation and/or Delete info we store in 
PRowImpl.
- Have a reasonable (smaller) default for the new 
QueryServices.MAX_MUTATION_SIZE_BYTES_ATTRIB

[~samarthjain] - would you mind giving this a quick look and committing if it 
looks fine?

> Make mutable batch size bytes-based instead of row-based
> --------------------------------------------------------
>
>                 Key: PHOENIX-541
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-541
>             Project: Phoenix
>          Issue Type: Improvement
>    Affects Versions: 3.0-Release
>            Reporter: mujtaba
>            Assignee: Geoffrey Jacoby
>              Labels: newbie
>             Fix For: 4.10.0
>
>         Attachments: PHOENIX-541.patch, PHOENIX-541-v2.patch
>
>
> With current configuration of row-count based mutable batch size, ideal value 
> for batch size is around 800 rather then current 15k when creating indexes 
> based on memory consumption, CPU and GC (data size: key: ~60 bytes, 14 
> integer column in separate CFs)



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

Reply via email to