Skip to content

Commit

Permalink
Remove redundant reconstruct call. (#2421)
Browse files Browse the repository at this point in the history
  • Loading branch information
robertnishihara authored and pschafhalter committed Jul 19, 2018
1 parent eed3916 commit 24eb140
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions python/ray/worker.py
Original file line number Diff line number Diff line change
Expand Up @@ -472,8 +472,9 @@ def get_object(self, object_ids):
# repeat.
while len(unready_ids) > 0:
for unready_id in unready_ids:
self.local_scheduler_client.reconstruct_objects(
[ray.ObjectID(unready_id)], False)
if not self.use_raylet:
self.local_scheduler_client.reconstruct_objects(
[ray.ObjectID(unready_id)], False)
# Do another fetch for objects that aren't available locally yet,
# in case they were evicted since the last fetch. We divide the
# fetch into smaller fetches so as to not block the manager for a
Expand All @@ -491,7 +492,7 @@ def get_object(self, object_ids):
self.local_scheduler_client.reconstruct_objects(
ray_object_ids_to_fetch[i:(
i + ray._config.worker_fetch_request_size())],
True)
False)
results = self.retrieve_and_deserialize(
object_ids_to_fetch,
max([
Expand Down

0 comments on commit 24eb140

Please sign in to comment.