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

Reply via email to