[ https://issues.apache.org/jira/browse/PHOENIX-3612?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16035163#comment-16035163 ]
James Taylor commented on PHOENIX-3612: --------------------------------------- +1. Looks really good, [~tdsilva]. > Make tracking of max allowed number of mutations bytes based instead of row > based > --------------------------------------------------------------------------------- > > Key: PHOENIX-3612 > URL: https://issues.apache.org/jira/browse/PHOENIX-3612 > Project: Phoenix > Issue Type: Bug > Reporter: James Taylor > Assignee: Thomas D'Silva > Fix For: 4.11.0 > > Attachments: PHOENIX-3612.patch > > > Some remaining work related to PHOENIX-541 to track the byte-size of all > mutations being buffered instead of the number of rows: > - 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 > This is essentially a guard on the memory usage. It could potentially > leverage our MemoryManager. -- This message was sent by Atlassian JIRA (v6.3.15#6346)