Hi Lewis,

I am not an HBase developer, but have done a lot of work with HBase over
the past years.
It looks rather dubious that there are 3 deletes like that, and it should
not be necessary unless there is some other race condition going on.

I'd suggest removing those, and if an issue arises I'd be happy to take a
look.

Cheers,
Tim




On Thu, Oct 8, 2015 at 2:07 PM, Lewis John Mcgibbney <
lewis.mcgibb...@gmail.com> wrote:

> Yes this should be handled by the datastore.
> I have a feeling that this was inherent behavior for hBase some time ago
> and it had changed and become more robust now.
> We should confirm from hBase devs.
>
>
> On Thursday, October 8, 2015, Renato Marroquín Mogrovejo <
> renatoj.marroq...@gmail.com> wrote:
>
>> Hi Lewis,
>>
>> I think that one makes sense,  we have three fields inserted, and we
>> delete the three of them. But still seems off, and that test being part of
>> core makes it even worst because this is not an HBase specific test. So for
>> every field that we added to the query object, we have to call delete?
>> shouldn't the dataStore do that by itself?
>>
>> 2015-10-08 2:40 GMT+02:00 Lewis John Mcgibbney <lewis.mcgibb...@gmail.com
>> >:
>>
>>> Hi Renato,
>>>
>>> On Wed, Oct 7, 2015 at 3:31 PM, Renato Marroquín Mogrovejo <
>>> renatoj.marroq...@gmail.com> wrote:
>>>
>>>>  I also don't understand why there are three table deletes.
>>>>
>>>
>>> did you ever notice
>>>
>>>
>>> https://github.com/apache/gora/blob/master/gora-core/src/test/java/org/apache/gora/store/DataStoreTestUtil.java#L1041-L1043
>>>
>>>
>>>
>>
>>
>
> --
> *Lewis*
>
>

Reply via email to