I don't think it's really unusual for deployment environments to produce
single shaded jars for an app.  Thus, I'm wondering if we can't rethink
this here?  E.g. just have a constant in code which states the version?
Rather than emit the confusing warning, especially for client apps?

Jason

On Fri, Jan 23, 2015 at 5:24 PM, Jun Rao <j...@confluent.io> wrote:

> The only impact is that you don't get the mbean that tells you the version
> of the jar. That's why it's just a warning.
>
> Thanks,
>
> Jun
>
> On Fri, Jan 23, 2015 at 1:04 PM, Jason Rosenberg <j...@squareup.com> wrote:
>
> > What are the ramifications if it can't find the version?  It looks like
> it
> > uses it set a yammer Gauge metric.  Anything more than that?
> >
> > Jason
> >
> > On Fri, Jan 23, 2015 at 3:24 PM, Jun Rao <j...@confluent.io> wrote:
> >
> > > Yes, that's probably the issue. If you repackage the Kafka jar, you
> need
> > to
> > > include the following in the repacked jar that was included in the
> > original
> > > Kafka jar. Our code looks for version info from there.
> > >
> > > META-INF/
> > >
> > > META-INF/MANIFEST.MF
> > >
> > > Thanks,
> > >
> > > Jun
> > >
> > > On Fri, Jan 23, 2015 at 11:59 AM, Jason Rosenberg <j...@squareup.com>
> > > wrote:
> > >
> > > > In this case, we have a single shaded jar for our app for deployment
> > (so
> > > > just 1 jar on the classpath).  Could that be the issue? E.g. all
> > > dependent
> > > > jars are unpacked into a single jar within our deployment system....
> > > >
> > > > On Thu, Jan 22, 2015 at 6:11 PM, Jun Rao <j...@confluent.io> wrote:
> > > >
> > > > > Hmm, kafka-console-consumer in 0.8.2 rc2 is running fine. Do you
> have
> > > > > multiple kafka jars in your classpath?
> > > > >
> > > > > Thanks,
> > > > >
> > > > > Jun
> > > > >
> > > > > On Thu, Jan 22, 2015 at 4:58 PM, Jason Rosenberg <j...@squareup.com
> >
> > > > wrote:
> > > > >
> > > > > > 2015-01-23 00:55:25,273  WARN [async-message-sender-0]
> > > common.AppInfo$
> > > > -
> > > > > > Can't read Kafka version from MANIFEST.MF. Possible cause:
> > > > > > java.lang.NullPointerException
> > > > > >
> > > > >
> > > >
> > >
> >
>

Reply via email to