HBASE-23264 Resolve a TODO of BucketAllocator: "Why we add the extra … (#801)

Signed-off-by: Viraj Jasani <vjasani@apache.org>
This commit is contained in:
bsglz 2020-04-26 17:45:31 +08:00 committed by Viraj Jasani
parent 9fc8263ca4
commit 178005efb9
No known key found for this signature in database
GPG Key ID: 3AE697641452FC5D
1 changed files with 2 additions and 1 deletions

View File

@ -274,8 +274,9 @@ public final class BucketAllocator {
// Default block size in hbase is 64K, so we choose more sizes near 64K, you'd better
// reset it according to your cluster's block size distribution
// The real block size in hfile maybe a little larger than the size we configured ,
// so we need add extra 1024 bytes for fit.
// TODO Support the view of block size distribution statistics
// TODO: Why we add the extra 1024 bytes? Slop?
private static final int DEFAULT_BUCKET_SIZES[] = { 4 * 1024 + 1024, 8 * 1024 + 1024,
16 * 1024 + 1024, 32 * 1024 + 1024, 40 * 1024 + 1024, 48 * 1024 + 1024,
56 * 1024 + 1024, 64 * 1024 + 1024, 96 * 1024 + 1024, 128 * 1024 + 1024,