Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

Filter SSE events - Round 2 #584

Closed
wants to merge 1 commit into from
Closed

Filter SSE events - Round 2 #584

wants to merge 1 commit into from

Conversation

paambaati
Copy link
Contributor

@paambaati paambaati commented May 10, 2018

This PR is a fresh copy of #452, and should help fix #435.

All credits to @toofishes for initially working on this.

@mesosphere-ci
Copy link

Can one of the admins verify this patch?

3 similar comments
@mesosphere-ci
Copy link

Can one of the admins verify this patch?

@mesosphere-ci
Copy link

Can one of the admins verify this patch?

@mesosphere-ci
Copy link

Can one of the admins verify this patch?

@mesosphere-ci
Copy link

Can one of the admins verify this patch?

1 similar comment
@mesosphere-ci
Copy link

Can one of the admins verify this patch?

@jkoelker
Copy link
Contributor

@paambaati Apologies for the delay in reviewing this patch.

I've fixed the flake8/import errors in commit a4270a8. If you'd like to incorporate those changes here to get it to pass flake8, I'll merge this pr in. Thanks for taking the time to fixup the older pr and resubmit it!

jkoelker added a commit that referenced this pull request Apr 4, 2019
* Fresh copy of #452

* mlb: flake8 fixes
@jkoelker
Copy link
Contributor

jkoelker commented Apr 4, 2019

We've merged this in via #624. Again apologies for the delay in getting to this and thanks for taking the time to make marathon-lb better!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use filtered SSE
3 participants