Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
Received-SPF: pass (athena.apache.org: domain of daan.hoogland@gmail.com
 designates 209.85.128.174 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <B2E3A407-5BC0-4DEB-A920-3641489A635A@basho.com>
References: <1A7163247D498044A96630B712400E10114BE2E8@SBPOMB101.sbp.lan>
 <51BA74E4.8000104@stratosphere.co.jp>
 <CAGQtxvaqkOAAjNrnB=ZgdMNH35smFw8Jxkn7xoCMLPfDGHvwwg@mail.gmail.com>
 <CAGQtxvZ3VFu+tw5O-95kUs9jpu2ScaCxExYzonaE-Sa-HkQxPw@mail.gmail.com>
 <B2E3A407-5BC0-4DEB-A920-3641489A635A@basho.com>
From: Daan Hoogland <daan.hoogland@gmail.com>
Date: Fri, 14 Jun 2013 17:52:28 +0200
Message-ID: 
 <CAGQtxvYS-adt9o7UEcd_UGFwL7qDqc=95D3XWz8iYRw64=aZEQ@mail.gmail.com>
Subject: Re: committer wanted for review
To: dev <dev@cloudstack.apache.org>
Content-Type: multipart/alternative; boundary=20cf3071cc482a768204df1f3d7a

--20cf3071cc482a768204df1f3d7a
Content-Type: text/plain; charset=ISO-8859-1

H John,

I browsed through your comments and most I will apply. There is one where
you contradict Hiroaki. This is about the logging level for reporting a
changed path. I am going to follow my heart at this unless there is a
project directive on it.

regards,
Daan


On Fri, Jun 14, 2013 at 5:25 PM, John Burwell <jburwell@basho.com> wrote:

> Daan,
>
> I just looked through the review request, and published my comments.
>
> Thanks,
> -John
>
> On Jun 14, 2013, at 10:27 AM, Daan Hoogland <daan.hoogland@gmail.com>
> wrote:
>
> > Hiroaki,
> >
> > - auto-fix may happen where it is really required
> >>
> > I do not have a clear view on this, so I took the approach of better safe
> > then sorry. The submitted is what works. I don't see how the auto-fix
> > should ever be needed if the source is fixed. Hope you can live with
> this.
> >
> >> - and if auto-fix happens, it should log it with
> >> WARN level.
> >
> > Applied
> >
> >
> > regards,
> >
> >
> > On Fri, Jun 14, 2013 at 10:35 AM, Daan Hoogland <daan.hoogland@gmail.com
> >wrote:
> >
> >> Thanks Hiroaki,
> >>
> >> On Fri, Jun 14, 2013 at 3:41 AM, Hiroaki KAWAI <
> kawai@stratosphere.co.jp>wrote:
> >>
> >>> I'd suggest:
> >>> - fix the generation of double slash itself
> >>>
> >> Is in the patch
> >>
> >>> - auto-fix may happen where it is really required
> >>> - and if auto-fix happens, it should log it with
> >>> WARN level.
> >>
> >> Good point, I will up the level in an update.
> >>
> >>>
> >>>
> >>>
> >>> (2013/06/13 21:15), Daan Hoogland wrote:
> >>>
> >>>> H,
> >>>>
> >>>> Can someone look at Review Request #11861<https://reviews.apache.**
> >>>> org/r/11861/ <https://reviews.apache.org/r/11861/>> for me please?
> >>>>
> >>>> Thanks,
> >>>> Daan Hoogland
> >>>>
> >>>>
> >>>
> >>
>
>

--20cf3071cc482a768204df1f3d7a--