Also note that its possible that later we may be wrapping the BlobStore instance. For example add a wrapper for monitioring and in such a case this equality condition might fail. A more stable fix would be to check with registered BlobStore weather it knows the given blobId or not. Chetan Mehrotra
On Wed, Jul 22, 2015 at 9:09 AM, Chetan Mehrotra <chetan.mehro...@gmail.com> wrote: > Hi Manfred, > > On Tue, Jul 21, 2015 at 10:58 PM, <bae...@apache.org> wrote: >> + if (bsbBlobStore != null && bsbBlobStore.equals(blobStore)) >> { >> + return bsb.getBlobId(); >> + } > > Can we have a testcase for this scenario? So far we do not have > requirement to support equality for BlobStore instance. So would like > to understand the usecase preferably with a testcase. May be the > problem affect SegmentNodeStore also (not sure though) > > Chetan Mehrotra