Skip to content
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

Revert "Revert "Remove superflous CheckReleased call which will slow down Event Log R…"" #36188

Merged
merged 1 commit into from
May 11, 2020

Conversation

tarekgh
Copy link
Member

@tarekgh tarekgh commented May 10, 2020

Reverts #36143

Per the comment, #34568 (comment) looks this change is not the direct cause of the failed test. We are reverting the revert again and applying the original fix. We should look at fixing the test or disabling it.

…down Event Log Reading up to 20x times. (#35911)" (#36143)"

This reverts commit ca8c9de.
@ghost
Copy link

ghost commented May 10, 2020

Tagging subscribers to this area: @ViktorHofer
Notify danmosemsft if you want to be subscribed.

@ghost
Copy link

ghost commented May 10, 2020

Tagging subscribers to this area: @tommcdon, @krwq
Notify danmosemsft if you want to be subscribed.

@tarekgh
Copy link
Member Author

tarekgh commented May 10, 2020

@safern could you please approve this one too.

@tarekgh tarekgh merged commit 35eb262 into master May 11, 2020
@safern safern deleted the revert-36143-revert-35911-eventlogperf_fix branch May 11, 2020 17:37
@ghost ghost locked as resolved and limited conversation to collaborators Dec 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants