Hi Allen,

Thanks for commenting,

I will update patch and keep this thread updated.

Wangda

On Thu, Feb 25, 2016 at 8:10 AM, Allen Wittenauer <[email protected]> wrote:

>
> > On Feb 24, 2016, at 5:10 PM, Wangda Tan <[email protected]> wrote:
> >
> >
> > Since YARN-3368 hasn't integrated to neither Hadoop's maven build system
> > nor RM web service, so the merge is completely safe. Ticket for the merge
> > task is: YARN-4734.
>
>
> -1 (non-binding)
>
> I don’t even have to run the build to see that:
>
> a) dot files in subdirectories that won’t work as intended
> b) ASF license violations all over the place
>
> This branch is definitely not ready to merge (the patch doesn’t even
> apply!) and I’m *blown away* a PMC member would even suggest it is ready.

Reply via email to