Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Avro, mail # dev - review recent C patches?


Copy link to this message
-
Re: review recent C patches?
Douglas Creager 2011-12-13, 19:04
> 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