[Gluster-devel] Some reviews/merges needed

Vijay Bellur vbellur at redhat.com
Fri Jan 9 17:49:36 UTC 2015


On 01/07/2015 09:10 PM, Xavier Hernandez wrote:
> Hi,
>
> I've some patches pending to be reviewed/merged. If someone has some
> time to look at it I'll appreciate it:
>
> http://review.gluster.org/8434/
>      A memory leak, though current code shouldn't trigger it. It's been
> also reviewed. Only merge needed.

Merged.

>
> http://review.gluster.org/9031/ (master)
> http://review.gluster.org/9032/ (release-3.6)
>      Fixes an spurious error that can cause crashes. Not sure if this
> has been addressed in another patch. If so I'll abandon it, otherwise
> this needs to be reviewed and merged.

I will be reviewing this later. Additional reviews are welcome.

>
> http://review.gluster.org/9079/
>      This fix is for master. The corresponding backport to release-3.6
> has already been merged. It has a +1.

Merged.

>
> http://review.gluster.org/9407/
>      Fixes a bug in ec that could cause EIO errors when some brick is
> damaged or replaced. I'll backport it to 3.6 once merged.

Dan & Pranith - can you please help with review of this one?

Thanks,
Vijay




More information about the Gluster-devel mailing list