Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/3798#issuecomment-72791220
Holy crap! Dont bother about this at all. This can wait. I hope everything
is okay. Take care and all the best!
On Feb 3, 2015 8:45 PM, "Cody Koeninger" <[email protected]> wrote:
> The warning is for metadata.broker.list, since its not expected by the
> existing ConsumerConfig (its used by other config classes)
>
> Couldn't get subclassing to work, the verifiedproperties class it uses is
> very dependent on order of operations during construction.
>
> I think the simplest thing is a class that is constructed using
> kafkaparams, and uses the static defaults from the ConsumerConfig object.
>
> I'm currently waiting in an ER with my child with a 105 fever, so won't be
> getting to it for a few hours to say the least.
> On Feb 3, 2015 10:15 PM, "Tathagata Das" <[email protected]> wrote:
>
> > I think the simplest solution is to assign zookeeper.connect. But you
are
> > assigning it in KafkaCluster lines 338 - 345. So why is this warning
> being
> > thrown?
> >
> > â
> > Reply to this email directly or view it on GitHub
> > <https://github.com/apache/spark/pull/3798#issuecomment-72787965>.
> >
>
> â
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/spark/pull/3798#issuecomment-72790044>.
>
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
