Emanuele Sabellico

Results 215 comments of Emanuele Sabellico

Hello Federico, thanks. I'm aware of the issue. Have done a fix in the .NET client but I'm going to do it for librdkafka too.

Yes, we've merged the fix [here](https://github.com/confluentinc/librdkafka/pull/4245) and are planning a maintenance release soon.

@lpsinger are you sure? I've just tried with a non existing topic and it gives ``` Traceback (most recent call last): File "consumer.py", line 98, in raise KafkaException(msg.error()) cimpl.KafkaException: KafkaError{code=UNKNOWN_TOPIC_OR_PART,val=3,str="Subscribed...

@lpsinger Thanks for that, I've installed and reproduced. The error happens in a different place than the initial one for a fix we did to the consume batch in 2.1.0....

@milibopp It doesn't seem the same issue, in that test `test_oauth_cb_principal_sasl_extensions` it's not subscribing to any topics. I couldn't reproduce it by running that test, could your provide some hint...

@lpsinger after reviewing all the cases only in `c_Node_to_py` it's needed to check for NULL values because it takes the nodes from the Metadata request and some of then can...

Thanks @dirkmueller @asottile-sentry ! Chose @dirkmueller one because `license_files` is available since setuptools 42 (Nov 2019) and it's older than 3 years. [Confluent Platinum end of support](https://docs.confluent.io/platform/current/clients/index.html#librdkafka-and-derived-clients)

@asottile-sentry the author is still @dirkmueller. We cannot merge directly because our CI won't run on external code and that code could change any moment, so for security reasons. We're...

We're working on a way to run CI directly on external PRs, still after manual review of its code, so we don't have to do this.