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 Plain View
Avro >> mail # dev >> review recent C patches?


+
Doug Cutting 2011-12-13, 18:02
Copy link to this message
-
Re: review recent C patches?
> A number of C patches have been submitted to Avro recently.
>
> http://s.apache.org/uo
>
> Can someone more familiar with the C implementation than I am please
> review these?  If no one else does first, I'll try to later this week.
>
> My approach would be to check that:
>  0. They sound and look reasonable.
>  1. They include a test.
>  2. Tests pass after they are applied.
> If these are true then I'll commit them.
>
> Others would probably be much better at (0) than I am.
>
> As a project we should review patches promptly.

My apologies for not getting to them sooner.  I'm taking a look at them now.

cheers
–doug

+
Doug Cutting 2011-12-13, 19:16
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