It seems to me the main blockers are: 1. DRILL-4547 Javadoc fails with Java8 <-- Can we split up the work among few people to resolve these ? 2. DRILL-6216 Metadata mismatch.. <-- Agreement was to revert one small piece of code and it appears Sorabh is looking into it 3. TestDrillbitResilience.cancelAfterAllResultsProduced <-- need someone to look into this
Regarding the JDBC issues that Parth mentioned, looking at the JIRAs, it seems they are not showstoppers...Parth do you agree ? Since we are close to the finish line for JDK 8, IMO we should try and see if in another day or two we can get over these hurdles. -Aman On Wed, Mar 7, 2018 at 7:17 AM, Pritesh Maker <pma...@mapr.com> wrote: > The JDK 8 issues will likely require more time to harden for it to be > included in the 1.13 release. My recommendation would be to move ahead with > the 1.13 release now and address these issues right. > > Pritesh > > -----Original Message----- > From: Parth Chandra <par...@apache.org> > Sent: March 7, 2018 3:34 AM > To: dev <dev@drill.apache.org> > Subject: Re: [DISCUSS] 1.13.0 release > > My mistake Volodymyr. > > Found some other JDK 8 issues in JIRA not tracked in DRILL-1491 > > DRILL-4547 Javadoc fails with Java8 > DRILL-6163 Switch Travis To Java 8 > > The following are tracked in DRILL-1491, but it doesn't look like we're > addressing these. Are we? > > DRILL-4329 13 Unit tests are failing with JDK 8 > DRILL-4333 DRILL-4329 tests in > Drill2489CallsAfterCloseThrowExceptionsTest fail in Java 8 > DRILL-5120 Upgrade JDBC Driver for new Java 8 methods > DRILL-5680 BasicPhysicalOpUnitTest can't run in Eclipse with Java 8 > > > *DRILL-4547 is a showstopper*. The release build (-Papache-release) fails > with far too many Javadoc errors even with doc lint turned off. > > DRILL-4333, DRILL-4329, DRILL-5120 are JDBC related which is a project by > itself. > > Note that fixing JDBC related issues and adding the command line option to > turn doc lint off will likely break Java 7 builds. > > > Folks who voted to get JDK 8 into this release, what is the consensus on > JDBC/Java8 ? > Also, any volunteers on helping debug > TestDrillbitResilience.cancelAfterAllResultsProduced > ? > > > > On Wed, Mar 7, 2018 at 3:20 PM, Volodymyr Tkach <vovatkac...@gmail.com> > wrote: > > > Addition to my last message: > > The link with PR for DRILL-1491 https://urldefense.proofpoint. > com/v2/url?u=https-3A__github.com_apache_drill_pull_1143&d=DwIBaQ&c= > cskdkSMqhcnjZxdQVpwTXg&r=zySISmkmM4WNViCKijENtQ&m=oTnKwfjj5hFBosMrq_ > WWhazhGeoC2nGSKeMOPxU2_cM&s=p3uialdRhgnf3XRY22R4SWXGZIq66aPijuy-Ms0J_-4&e= > > on which the we can see TestDrillbitResilience. > > cancelAfterAllResultsProduced > > failure. > > > > 2018-03-07 11:45 GMT+02:00 Volodymyr Tkach <vovatkac...@gmail.com>: > > > > > *To Parth:* > > > The failure can only be seen if run on DRILL-1491 branch, because it > uses > > > jdk 1.8 in pom.xml > > > > > > <source>1.8</source> > > > <target>1.8</target> > > > > > > 2018-03-07 6:03 GMT+02:00 Sorabh Hamirwasia <shamirwa...@mapr.com>: > > > > > >> Just sent an email on RCA of DRILL-6216 to discuss next steps. > > >> > > >> > > >> Thanks, > > >> Sorabh > > >> > > >> ________________________________ > > >> From: Parth Chandra <par...@apache.org> > > >> Sent: Tuesday, March 6, 2018 6:48:21 PM > > >> To: dev > > >> Subject: Re: [DISCUSS] 1.13.0 release > > >> > > >> We have two items remaining - > > >> > > >> DRILL-1491 - Ideally, I would like to make sure that CANCEL is handled > > >> correctly with JDK 8. If the failure of the unit test is because the > > >> cancel > > >> is received after the query is completed, then the issue is less > severe, > > >> but I would like to be sure that this is the case. > > >> Are there others who see the DrillbitResilience tests failing for > them? > > >> Can > > >> we try to assist Volodymyr? I don't see the failures myself. > > >> > > >> DRILL-6216 - this is a showstopper. > > >> > > >> > > >> On Wed, Mar 7, 2018 at 5:27 AM, Kunal Khatua <kunalkha...@gmail.com> > > >> wrote: > > >> > > >> > Hi Parth > > >> > > > >> > DRILL-6216 is a release blocker that is being currently looked into. > > >> > > > >> > Ref: > > >> > DRILL-6216: Metadata mismatch when connecting to a Drill 1.12.0 > with a > > >> > Drill-1.13.0-SNAPSHOT driver > > >> > https://urldefense.proofpoint.com/v2/url?u=https-3A__issues. > > >> apache.org_jira_browse_DRILL-2D6216&d=DwIBaQ&c=cskdkSMqhcnjZ > > >> xdQVpwTXg&r=gRpEl0WzXE3EMrwj0KFbZXGXRyadOthF2jlYxvhTlQg&m=xu > > >> Rz02Sbprxvbtw1OrBuDvlRbp2lh9mz3sxpP5-wHPs&s=txeKaKzF67flAi48 > > >> DUNLgMWbxje1GXWxfFpG6BEPXk0&e= > > >> > > > >> > Please add it to the list of required commits as well. > > >> > > > >> > Thanks > > >> > ~ Kunal > > >> > On 3/6/2018 9:53:06 AM, Volodymyr Tkach <vovatkac...@gmail.com> > > wrote: > > >> > Right now i haven't found the reason of > > >> > TestDrillbitResilience.cancelAfterAllResultsProduced failure, most > > >> likely > > >> > the cause of the failure is that the query is able to have been > > >> completed > > >> > before cancellation request is processed. > > >> > This test not only the case, there is one more ignored test > > >> > TestDrillbitResilience.cancelAfterEverythingIsCompleted and jira > > >> > DRILL-3967 > > >> > created, although the environment is AWS. > > >> > > > >> > Maybe it makes sense to ignore this test to unblock the release and > > >> merge > > >> > JDK8 changes? > > >> > > > >> > > > > > > > > >