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.

Sign up for GitHub

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

Jump to bottom

List/Search results blocking until completion instead of streaming output #143

Closed
ferventcoder opened this issue Mar 3, 2015 · 0 comments
Closed

List/Search results blocking until completion instead of streaming output #143

ferventcoder opened this issue Mar 3, 2015 · 0 comments
Assignees
@ferventcoder
Labels
0 - Backlog Bug Priority_HIGH
Milestone
0.9.9.9

Comments

@ferventcoder
Copy link
Member

226e998 removed yield returns so one must wait for the entire list to return before seeing output.
@ferventcoder ferventcoder modified the milestones: 0.9.9.1, 0.9.9.2 Mar 3, 2015
Jaykul added a commit to Jaykul/choco that referenced this issue Mar 4, 2015
@Jaykul
(chocolateyGH-143) Make NuGet calls yield return, so we stream output…
dc71a47 
… as it comes
@ferventcoder ferventcoder modified the milestones: 0.9.9.2, 0.9.9.3 Mar 7, 2015
@ferventcoder ferventcoder modified the milestones: 0.9.9.3, 0.9.9.5 Mar 30, 2015
@ferventcoder ferventcoder modified the milestones: 0.9.9.5, 0.9.9.6 Apr 20, 2015
@ferventcoder ferventcoder modified the milestones: 0.9.9.6, 0.9.9.7 May 15, 2015
@ferventcoder ferventcoder modified the milestones: 0.9.9.7, 0.9.9.8 Jun 12, 2015
@ferventcoder ferventcoder removed this from the 0.9.9.8 milestone Jun 26, 2015
@ferventcoder ferventcoder modified the milestones: 0.9.9.9, 0.9.9.8 Jun 26, 2015
ferventcoder pushed a commit that referenced this issue Jun 29, 2015
@Jaykul @ferventcoder
(GH-143) Ensure NuGetList streams output
a816de1 
Remove ToList() in the calls to NuGetList.GetPackages() and return an
IEnumerable to ensure that calls to List stream output instead of performing
blocking operations to return results from searching package feeds.
ferventcoder added a commit that referenced this issue Jun 29, 2015
@ferventcoder
Merge branch 'Stream_Results' into stable
73ba21f 
* Stream_Results:
  (GH-143) Ensure NuGetList streams output
ferventcoder added a commit that referenced this issue Jun 29, 2015
@ferventcoder
Merge branch 'stable'
79ed3af 
Update all of the ISourceRunners with changes to
IEnumerable<PackageResult>.

* stable:
  (GH-143) Ensure NuGetList streams output

Conflicts:
	src/chocolatey/infrastructure.app/services/INugetService.cs
	src/chocolatey/infrastructure.app/services/NugetService.cs
@Jaykul Jaykul mentioned this issue Jun 29, 2015
Closed
@ferventcoder ferventcoder changed the title ToList on Nuget list command is bad, mmkay? Search results blocking until completion - NuGet using ToList too early Jul 25, 2015
@ferventcoder ferventcoder changed the title Search results blocking until completion - NuGet using ToList too early Search results blocking until completion instead of streaming Jul 25, 2015
@ferventcoder ferventcoder changed the title Search results blocking until completion instead of streaming Search results blocking until completion instead of streaming output Jul 25, 2015
@ferventcoder ferventcoder changed the title Search results blocking until completion instead of streaming output List/Search results blocking until completion instead of streaming output Jul 25, 2015
@ferventcoder ferventcoder self-assigned this Sep 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ferventcoder ferventcoder

Labels
0 - Backlog Bug Priority_HIGH
Projects
None yet
Milestone
0.9.9.9
Development

Successfully merging a pull request may close this issue.

(GH-143) Make NuGet calls yield return, so we stream output as it comes Jaykul/choco
1 participant
@ferventcoder