-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
1.4.7 #1903
Comments
There are two things on my mind: (1) consumer performance degradation in
#1888 . I have attempted to fix w
#1902 but it is a significant
internal change and so has risk. I haven't decided whether to make this the
default and allow opt out or make it opt in and wait for a 1.5 release to
make it the default.
(2) Heartbeat thread coordination / deadlocks / timeouts remain an issue.
One structural change that I have deferred is in #1821 . I am undecided
whether to make this change now or in the future after thinking more about
the deadlock issue I mention in the comments.
(3) I would like to make some changes to the network write IO so that we
use non blocking socket send() with a buffer and also improve error
handling. I have done some work on this already but haven't put up a PR.
If you'd like to pitch in, maybe just give me your thoughts on these
issues. How would you manage them?
…On Wed, Sep 18, 2019, 10:32 AM Ofek Lev ***@***.***> wrote:
@dpkp <https://github.com/dpkp> Hello there! Are there any blockers for
releasing 1.4.7? If so, can I help?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1903?email_source=notifications&email_token=AAGN5NHV5YX2BFFBVHU6N4DQKJQ3PA5CNFSM4IYBWBA2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HMGTKQQ>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGN5NEJ7F5OWS5ZYE7X4E3QKJQ3PANCNFSM4IYBWBAQ>
.
|
Also I need to learn to count to three.
|
@dpkp Thanks for the thoughtful response!
However, you should know I'm biased for 2 reasons:
|
What do you think? |
Thanks for pushing on this @ofek. I've also tried to get @jeffwidman and @dpkp to cut a patch release since Aug 16 to include my PR #1883 as it resolves handshake failures for unknown root CAs such as Amazon's that are blocking downstream projects such as Datadog's Kafka consumer integration. |
@dpkp Any progress on this? |
Hi team - I'll take a stab this weekend!
…On Wed, Sep 25, 2019, 9:45 PM Ofek Lev ***@***.***> wrote:
@dpkp <https://github.com/dpkp> Any progress on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1903?email_source=notifications&email_token=AAGN5NAT7QVDIOAEGCEOETTQLQ45ZA5CNFSM4IYBWBA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7UIQJA#issuecomment-535332900>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGN5NFZCRBWTU2WMGGXQ5LQLQ45ZANCNFSM4IYBWBAQ>
.
|
@dpkp Great, thanks! |
1.4.7 released + uploaded to pypi. Please report any issues! |
@dpkp Thank you so much! |
Thanks for all your hard work on this @dpkp! |
@dpkp Hello there! Are there any blockers for releasing
1.4.7
? If so, can I help?The text was updated successfully, but these errors were encountered: