Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Threaded View
HBase >> mail # dev >> Return value from RegionObserver.preIncrement()


Copy link to this message
-
Re: Return value from RegionObserver.preIncrement()
Hi Daniel,

If you like, the best way would be to provide a patch :)

Lars
On Nov 30, 2011, at 3:47 PM, Daniel Gómez Ferro wrote:

> Hi Lars,
>
> Done: https://issues.apache.org/jira/browse/HBASE-4903
>
> What's the best option to solve this?
>
> Thanks,
> Daniel
>
> On Nov 30, 2011, at 15:10 , Lars George wrote:
>
>> Hi Daniel,
>>
>> This indeed looks like an oversight. Could you create a JIRA for it please?
>>
>> Thanks,
>> Lars
>>
>> On Nov 30, 2011, at 12:37 PM, Daniel Gómez Ferro wrote:
>>
>>> Hi,
>>>
>>> RegionObserver.preIncrement() receives a Result object which it can modify if it wants to return directly to the client. However the only way of modifying it that I could find is through Result.readFields(). That would require the user to create the Result object he wants, serializing it and then deserializing.
>>>
>>> I think it would be better to either change preIncrement() to return a Result object or add a setter to Result. We could also change preIncrement() to receive a List<KeyValue> like preGet() does.
>>>
>>> Thanks.
>>>
>>> Daniel
>>
>
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB