Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Accumulo, mail # dev - FooConstraint.jar Source


Copy link to this message
-
Re: FooConstraint.jar Source
Keith Turner 2013-11-20, 17:48
On Wed, Nov 20, 2013 at 12:30 PM, Mike Drob <[EMAIL PROTECTED]> wrote:

> Does not having the source run us afoul of ASF policy?
>

I don't know, but I agree w/ Josh that it should be fixed before release.
 At the very least the Constraint jar should have source like the Filter
jar.   I used javap to disassemble FooFitler.class and I think it lines up
w/ the source.
>
>
> On Wed, Nov 20, 2013 at 12:25 PM, Josh Elser <[EMAIL PROTECTED]> wrote:
>
> > Yup to both. We definitely need to address this before we make the next
> > release of Accumulo.
> >
> >
> > On 11/20/13, 11:44 AM, Keith Turner wrote:
> >
> >> On Wed, Nov 20, 2013 at 11:33 AM, Sean Busbey <
> busbey+[EMAIL PROTECTED]
> >> >wrote:
> >>
> >>  Sounds like we need a ticket to reimplement that class.
> >>>
> >>> Fair to mark as newbie?
> >>>
> >>>
> >> Creating the source is certainly trivial.  Making maven build the jar
> for
> >> the test, I am not sure how easy that is.  It certainly needs a ticket.
> >>
> >>
> >>
> >>>
> >>> On Wed, Nov 20, 2013 at 9:32 AM, Keith Turner <[EMAIL PROTECTED]>
> wrote:
> >>>
> >>>  I don't think the source was ever in source control.  Just the jar.
> >>>>
> >>>>
> >>>> On Tue, Nov 19, 2013 at 10:31 PM, Josh Elser <[EMAIL PROTECTED]>
> >>>>
> >>> wrote:
> >>>
> >>>>
> >>>>  I'm working through ShellServerTest and noticed we have two jars that
> >>>>>
> >>>> are
> >>>
> >>>> used for testing.
> >>>>>
> >>>>> 'test/src/test/resources/FooFilter.jar' contains a class file and the
> >>>>> java source file, but 'test/src/test/resources/FooConstraint.jar'
> only
> >>>>> contains a class file.
> >>>>>
> >>>>> Is the source for this class somewhere that I missed? I can't find it
> >>>>> elsewhere in the tree.
> >>>>>
> >>>>>
> >>>>
> >>>
> >>>
> >>> --
> >>> Sean
> >>>
> >>>
> >>
>