Gopal Holla

Saw an exception in `FrontendIntegrationTest` during a travis build. Seems to be intermittent but filing this issue to keep track since 1) This is in the `stitchBlob` test (recently introduced)...

VerifiableProperties needs more APIs

2

`VerifiableProperties` needs a few more APIs to support the use cases that have cropped. - a "get if positive, otherwise throw" function for all types - also provide separate functions...

Make names of variables and functions consistent in PersistentIndex

3

`PersistentIndex.java` uses index and index segment interchangeably to refer to index segments. This creates confusion and inconsistency and affects readability. Change `PersistentIndex.java` and `IndexSegment.java` so that all variable and function...

Currently Ambry sprays the data chunks of a given "large" blob across all writable partitions (i.e. any partition is eligible to receive a data chunk). This can create some problems...

``` com.github.ambry.router.ChunkFillTest > testChunkFillingBlobSizeNotMultipleOfChunkSize[1] FAILED junit.framework.AssertionFailedError: Chunk should be ready. expected: but was: at junit.framework.Assert.fail(Assert.java:47) at junit.framework.Assert.failNotEquals(Assert.java:283) at junit.framework.Assert.assertEquals(Assert.java:64) at com.github.ambry.router.ChunkFillTest.fillChunksAndAssertSuccess(ChunkFillTest.java:289) at com.github.ambry.router.ChunkFillTest.testChunkFillingBlobSizeNotMultipleOfChunkSize(ChunkFillTest.java:121) ```

Add an implementation of `AccountService` that can load account details from a json file on disk. Will help when helix is not required and for testing.

Allow compaction admin requests to specify policy to use

2

Currently, the admin trigger of compaction uses the preconfigured compaction policy that has been set in the store config. It will be useful for the admin request to optionally specify...

Explore ways to trigger compaction based on the how "full" the store is. There should be a mechanism to request compaction (akin to how admin requests to trigger compaction are...

``` com.github.ambry.account.AccountUpdateToolTest > testUpdateAccount FAILED org.I0Itec.zkclient.exception.ZkTimeoutException: Unable to connect to zookeeper server within timeout: 2000 at org.I0Itec.zkclient.ZkClient.connect(ZkClient.java:1120) at org.I0Itec.zkclient.ZkClient.(ZkClient.java:147) at org.I0Itec.zkclient.ZkClient.(ZkClient.java:122) at org.apache.helix.manager.zk.ZkClient.(ZkClient.java:60) at org.apache.helix.manager.zk.ZkClient.(ZkClient.java:69) at org.apache.helix.manager.zk.ZkClient.(ZkClient.java:82) at com.github.ambry.commons.HelixNotifier.(HelixNotifier.java:75) at...

BlobStoreStats returns valid log segment size map in the wrong order

1

`BlobStoreStats` has a function to get size by log segment. The return type is a `NavigableMap` but the sort order in the map is incorrect. This causes `DefaultCompactionPolicy` to pick...