[
https://issues.apache.org/jira/browse/HBASE-22412?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Zheng Hu updated HBASE-22412:
-----------------------------
Description:
Address the comment in HBASE-22387:
bq. The ByteBuffAllocator#getFreeBufferCount will be O(N) complexity, because
the buffers here is an ConcurrentLinkedQueue. It's worth file an issue for this.
Also I think we should use the allcated bytes instead of allocation number to
evaluate the head allocation percent , so that we can decide whether the
ByteBuffer is too small and will have higher GC pressure.
If the heap allocation percent is less than
hbase.ipc.server.reservoir.minimal.allocating.size /
hbase.ipc.server.allocator.buffer.size, then the allocator works fine,
otherwise it's overload.
was:
Address the comment in HBASE-22387:
bq. The ByteBuffAllocator#getFreeBufferCount will be O(N) complexity, because
the buffers here is an ConcurrentLinkedQueue. It's worth file an issue for this.
Also I think we should use the allcated bytes instead of allocation number to
evaluate the head allocation percent , so that we can decide whether the
ByteBuffer is too small and will have higher GC pressure.
> Improve the metrics in ByteBuffAllocator
> ----------------------------------------
>
> Key: HBASE-22412
> URL: https://issues.apache.org/jira/browse/HBASE-22412
> Project: HBase
> Issue Type: Sub-task
> Reporter: Zheng Hu
> Assignee: Zheng Hu
> Priority: Major
>
> Address the comment in HBASE-22387:
> bq. The ByteBuffAllocator#getFreeBufferCount will be O(N) complexity, because
> the buffers here is an ConcurrentLinkedQueue. It's worth file an issue for
> this.
> Also I think we should use the allcated bytes instead of allocation number to
> evaluate the head allocation percent , so that we can decide whether the
> ByteBuffer is too small and will have higher GC pressure.
> If the heap allocation percent is less than
> hbase.ipc.server.reservoir.minimal.allocating.size /
> hbase.ipc.server.allocator.buffer.size, then the allocator works fine,
> otherwise it's overload.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
