ExecuteAsync creates unnecessary Task

kelvinRosa / google-api-dotnet-client

Automatically exported from code.google.com/p/google-api-dotnet-client

Apache License 2.0

0 stars 0 forks source link

ExecuteAsync creates unnecessary Task #383

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

We should change the implementation of ExecuteAsync and apparently of 
MediaUpload and MediaDownload. 
We should not create a new task. It’s just a waste of task. The current task 
will return to the thread pool tasks when we will make the call to the  
network. So there is no need in additional task

Original issue reported on code.google.com by pele...@google.com on 22 Aug 2013 at 1:46

GoogleCodeExporter commented 9 years ago

Original comment by pele...@google.com on 22 Aug 2013 at 1:46

Added labels: Priority-High
Removed labels: Priority-Medium

GoogleCodeExporter commented 9 years ago

Original comment by pele...@google.com on 27 Sep 2013 at 6:50

Added labels: Milestone-Release1.6
Removed labels: Milestone-Release1.7

GoogleCodeExporter commented 9 years ago

Original comment by pele...@google.com on 1 Oct 2013 at 5:28

Removed labels: Milestone-Release1.6

GoogleCodeExporter commented 9 years ago

Original comment by pele...@google.com on 1 Oct 2013 at 7:12

Added labels: Milestone-Release1.6

GoogleCodeExporter commented 9 years ago

Original comment by pele...@google.com on 1 Oct 2013 at 7:31

Changed title: ExecuteAsync creates unnecessary Task

GoogleCodeExporter commented 9 years ago

https://codereview.appspot.com/14229044/

Original comment by pele...@google.com on 1 Oct 2013 at 7:32

Changed state: Started

GoogleCodeExporter commented 9 years ago

Original comment by pele...@google.com on 2 Oct 2013 at 1:48

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Original comment by pele...@google.com on 3 Oct 2013 at 7:28