Sorry for the delay. Based on discussion here, I merged HADOOP-9613 to trunk after the reviews. Thank you for the great helps. The problems reported on HADOOP-9613 by Wangda, Li, and Sunil is filed as YARN-5275. I'll also take a look at it.
I believe this change is a great step for compiling Hadoop source code with JDK8. Again, thanks a lot! Best, - Tsuyoshi On Thu, May 19, 2016 at 1:55 AM, Li Lu <[email protected]> wrote: > Sure. Sorry I got disturbed by some other fixes… I can check it today and > will update the outcome in the original JIRA (by the EOD today, pacific > time). At the same time, YARN-3368 (new UI branch) may also need some tests? > > Li Lu > > On May 18, 2016, at 09:08, Sangjin Lee <[email protected]> wrote: > > Li Lu was looking into this. Li, were you able to take a look? > > Thanks, > Sangjin > > On Wed, May 18, 2016 at 7:44 AM, Tsuyoshi Ozawa <[email protected]> wrote: >> >> Thanks for your help, Sangjin. Do you have any concern or problem >> after your testing? I'd like to merge it if you're okay. >> >> I'll wait for your reply until today - after raising this issue to ML, >> we had a time longer than 1 week, I think it's time to merge. >> >> Thanks, >> - Tsuyoshi >> >> On Sat, May 14, 2016 at 9:24 AM, Sangjin Lee <[email protected]> wrote: >> > We're also checking from the timeline service v.2 perspective >> > (YARN-2928). >> > >> > On Fri, May 13, 2016 at 3:33 PM, Tsuyoshi Ozawa <[email protected]> >> > wrote: >> >> >> >> Hi YARN developers, >> >> >> >> Could you check this thread? I think YARN web UI is affected by this >> >> change. >> >> >> >> Thanks, >> >> - Tsuyoshi >> >> >> >> ---------- Forwarded message ---------- >> >> From: Tsuyoshi Ozawa <[email protected]> >> >> Date: Mon, May 9, 2016 at 12:12 PM >> >> Subject: [DISCUSS] Merge Jersey upgrade(HADOOP-9613) to trunk >> >> To: "[email protected]" <[email protected]>, >> >> "[email protected]" <[email protected]>, >> >> "[email protected]" <[email protected]>, >> >> "[email protected]" <[email protected]> >> >> >> >> >> >> Hi developers, >> >> >> >> We’ve worked on upgrading jersey(HADOOP-9613) for a years. It's >> >> essential change to support complication with JDK8. It’s almost there. >> >> >> >> One concern to merge this to trunk is incompatibility. After the >> >> release of Jersey 1.13, the root element whose content is empty >> >> collection is changed from null to empty object({}). Because of this >> >> problem, I’ve marked HADOOP-9613 as incompatible change. Is it >> >> acceptable for us? If it’s acceptable change in trunk, I’d like to >> >> merge it into trunk. >> >> >> >> Thanks, >> >> - Tsuyoshi >> >> >> >> --------------------------------------------------------------------- >> >> To unsubscribe, e-mail: [email protected] >> >> For additional commands, e-mail: [email protected] >> >> >> > > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
