Skip to content
This repository was archived by the owner on Apr 4, 2023. It is now read-only.

Bugfix/dvpl 7185 develop #76

Merged
merged 5 commits into from
Feb 12, 2018
Merged

Bugfix/dvpl 7185 develop #76

merged 5 commits into from
Feb 12, 2018

Conversation

wimcolgate
Copy link
Contributor

@wimcolgate wimcolgate commented Feb 9, 2018

This PR is to work around the memory leak by replacing ExandoObject with a Dictionary. It's less pretty, but doesn't leak memory (inherent in ExpandoObject).

@@ -42,7 +42,7 @@ static void Main(string[] args)

//// Search : Export Previews

using (SearchPreviewStream stream = await service.ExportSearchPreviewsAsync("search index=_internal | head 100"))
using (SearchPreviewStream stream = await service.ExportSearchPreviewsAsync("search index=_internal"))
{
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "| head 100" needs to be put back in.

@ljiang1 ljiang1 changed the base branch from master to develop February 9, 2018 20:58
@ljiang1
Copy link
Contributor

ljiang1 commented Feb 9, 2018

The changes look good

@ljiang1 ljiang1 merged commit febfe04 into develop Feb 12, 2018
@shakeelmohamed shakeelmohamed deleted the bugfix/DVPL-7185-develop branch April 16, 2018 17:14
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.

2 participants