Mailing-List: contact hdfs-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: hdfs-dev@hadoop.apache.org
Received-SPF: pass (nike.apache.org: domain of adi@cloudera.com designates
 209.85.160.176 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAJs-t7O+ih56JG=q+Z83uWUZLitGU2Wm5N60xmypqLHxpLKKVA@mail.gmail.com>
References: <1340947673.32328.YahooMailNeo@web121702.mail.ne1.yahoo.com>
	<C1D18570-FB13-4542-B84B-92F7ED02AE72@hortonworks.com>
	<CAJs-t7M7y9MypJL5nJK0L_bF2ou-yRYPO61nx8rBQ_7KPbFAKQ@mail.gmail.com>
	<CAE_FxDie0v6YzEnmNecnqXJymB9z75iXaYfAFtDUQVabtEd1+g@mail.gmail.com>
	<CAE_FxDju4dJt75ZMC7ddyO0QfSNinPv35KW0W=PP=h7Jj6PPMQ@mail.gmail.com>
	<CAJs-t7N+W2o7rwmaj97QaaxJ4+g5E=dVGe0-vxDGJWYmESMM7Q@mail.gmail.com>
	<1341010190.13119.YahooMailNeo@web121702.mail.ne1.yahoo.com>
	<CAJs-t7O+ih56JG=q+Z83uWUZLitGU2Wm5N60xmypqLHxpLKKVA@mail.gmail.com>
Date: Fri, 29 Jun 2012 17:12:14 -0700
Message-ID: 
 <CAE_FxDhpO0dPcUDxjcROjk1HzqBPtB93pPN5xML6uk4zuHQZiQ@mail.gmail.com>
Subject: Re: Hadoop-2 generics compilation error
From: Andy Isaacson <adi@cloudera.com>
To: hdfs-dev@hadoop.apache.org
Cc: lars hofhansl <lhofhansl@yahoo.com>
Content-Type: multipart/alternative; boundary=14dae9340e29be9c5004c3a56a91

--14dae9340e29be9c5004c3a56a91
Content-Type: text/plain; charset=ISO-8859-1

Thanks everybody!  Good to get this resolved.

-andy

On Fri, Jun 29, 2012 at 3:53 PM, Alejandro Abdelnur <tucu@cloudera.com>wrote:

> yep, just tested with sunjdk, and it works. We still have the
> unboxing, but is done when the variable is used as parameter in method
> that is expecting a primitive type.
>
>
>
> On Fri, Jun 29, 2012 at 3:49 PM, lars hofhansl <lhofhansl@yahoo.com>
> wrote:
> > SunJDK will correctly infer the reference types, correct?
> > If so, changing this to reference types should make both JDKs happy.
> >
> > -- Lars
> >
> > ________________________________
> > From: Alejandro Abdelnur <tucu@cloudera.com>
> > To: hdfs-dev@hadoop.apache.org
> > Sent: Friday, June 29, 2012 3:26 PM
> > Subject: Re: Hadoop-2 generics compilation error
> >
> > Andy,
> >
> > It looks like OpenJDK javac does not handle type unboxing with a
> > generic method where the parameter that resolves the generic infers
> > the type to do the un-boxing on. While SunJDK does handle it.
> >
> > Now, to the question which one is correct? I'm not sure but SunJDK
> > does a valid inference.
> >
> > thx
> >
> > On Fri, Jun 29, 2012 at 2:43 PM, Andy Isaacson <adi@cloudera.com> wrote:
> >> I've filed https://issues.apache.org/jira/browse/HDFS-3580 and
> attached the
> >> patch.
> >>
> >> I'd really appreciate it if someone could explain what the problem is,
> why
> >> the patch fixes it, and whether this is a bug in OpenJDK (that it does
> not
> >> accept a valid program) or a bug in Sun JDK (that it does accept an
> >> incorrect program).
> >>
> >> -andy
> >>
> >> On Fri, Jun 29, 2012 at 12:01 PM, Andy Isaacson <adi@cloudera.com>
> wrote:
> >>
> >>> I ran into this as well, and am prepping a patch. Lars, if you could
> file
> >>> the Jira explaining why s/boolean/Boolean/ is the right fix, I'll test
> and
> >>> submit the patch.
> >>>
> >>> -andy
> >>>
> >>>
> >>> On Fri, Jun 29, 2012 at 11:08 AM, Alejandro Abdelnur <
> tucu@cloudera.com>wrote:
> >>>
> >>>> This change is from HDFS-3481, the wrong JIRA number is because I made
> >>>> a mistake in the commit message.
> >>>>
> >>>> I can put a patch to fix this, but don't have OpenJDK to test it. It
> >>>> would be better if somebody with OpenJDK takes care of it and I'll
> >>>> review it.
> >>>>
> >>>> Thx
> >>>>
> >>>> On Fri, Jun 29, 2012 at 11:01 AM, Arun C Murthy <acm@hortonworks.com>
> >>>> wrote:
> >>>> > HDFS-3113 is committed? Or, are you running with the patch yourself?
> >>>> >
> >>>> > It's still PA and I see Daryn reviewing it yet...
> >>>> >
> >>>> > Arun
> >>>> >
> >>>> > On Jun 28, 2012, at 10:27 PM, lars hofhansl wrote:
> >>>> >
> >>>> >> Since HDFS-3113 was integrated into Hadoop-2 I get the compilation
> >>>> errors of the following type:
> >>>> >>
> >>>> >> [ERROR]
> >>>>
> /home/lars/dev/hadoop-2/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/main/java/org/apache/hadoop/fs/http/server/HttpFSServer.java:[407,36]
> >>>> incompatible types; no instance(s) of type variable(s) V exist so
> that V
> >>>> conforms to boolean
> >>>> >>
> >>>> >>
> >>>> >> Indeed at line 407 I see:
> >>>> >>
> >>>> >>         boolean hasData = params.get(DataParam.NAME,
> DataParam.class);
> >>>> >>
> >>>> >>
> >>>> >> When I change that to
> >>>> >>         Boolean hasData = params.get(DataParam.NAME,
> DataParam.class);
> >>>> >>
> >>>> >>
> >>>> >> instead (along with long to Long, short to Short, etc, later in
> that
> >>>> file), everything compiles fine.
> >>>> >>
> >>>> >> $ javac -version
> >>>> >> javac 1.6.0_24
> >>>> >>
> >>>> >>
> >>>> >> $ java -version
> >>>> >> java version "1.6.0_24"
> >>>> >> OpenJDK Runtime Environment (IcedTea6 1.11.3)
> >>>> (fedora-67.1.11.3.fc16-x86_64)
> >>>> >> OpenJDK 64-Bit Server VM (build 20.0-b12, mixed mode)
> >>>> >>
> >>>> >> Apparently OpenJDK gets mixed up on the type inference here. Can we
> >>>> change this in the way I suggest so that Hadoop-2 (and presumably
> trunk)
> >>>> can be
> >>>> >> compiled with OpenJDK?
> >>>> >> I'm happy to create a jira and (trivial) patch.
> >>>> >>
> >>>> >>
> >>>> >> Thanks.
> >>>> >>
> >>>> >> -- Lars
> >>>> >>
> >>>> >
> >>>> > --
> >>>> > Arun C. Murthy
> >>>> > Hortonworks Inc.
> >>>> > http://hortonworks.com/
> >>>> >
> >>>> >
> >>>>
> >>>>
> >>>>
> >>>> --
> >>>> Alejandro
> >>>>
> >>>
> >>>
> >
> >
> >
> > --
> > Alejandro
>
>
>
> --
> Alejandro
>

--14dae9340e29be9c5004c3a56a91--