Skip to content

Commit

Permalink
[FLR] Use the correct Options when populating Executor::Args.
Browse files Browse the repository at this point in the history
My recent change modified how `Executor::Args` was populated and missed the fact that we
rewrite the `Options` to add in a created `Rendezvous` object in some cases. This change correctly uses the rewritten `Options` in both cases.

PiperOrigin-RevId: 220218606
  • Loading branch information
mrry authored and tensorflower-gardener committed Nov 6, 2018
1 parent 70ccf42 commit e436475
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tensorflow/core/common_runtime/function.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1028,9 +1028,9 @@ void FunctionLibraryRuntimeImpl::Run(const Options& opts, Handle handle,
}

Executor::Args exec_args;
ExecutorArgsFromOptions(opts, frame, &exec_args);
ExecutorArgsFromOptions(run_opts, frame, &exec_args);

bool allow_dead_tensors = opts.allow_dead_tensors;
bool allow_dead_tensors = run_opts.allow_dead_tensors;
item->exec->RunAsync(
// Executor args
exec_args,
Expand Down Expand Up @@ -1085,7 +1085,7 @@ void FunctionLibraryRuntimeImpl::Run(const Options& opts, Handle handle,
DCHECK(run_opts.runner != nullptr);

Executor::Args exec_args;
ExecutorArgsFromOptions(opts, frame, &exec_args);
ExecutorArgsFromOptions(run_opts, frame, &exec_args);
item->exec->RunAsync(exec_args, std::move(done));
}

Expand Down

0 comments on commit e436475

Please sign in to comment.