Change behavior of DirectSession so that it places the entire graph

by default, instead of only the pruned graphs.

Several reasons motivate this change:

- A bug in a the graph (program) today is only identified when
  you run the op.

- Interactive sessions need to allow graphs that have "bugs" in it
  because people just continue to add to their existing graph,
  so we need a way to preserve the old behavior.  We do that
  by setting the field by default to true in InteractiveSession.

Tests added for both the change in direct_session as well as
the InteractiveSession use case.

Fixes #1914, #1748

This refactors direct_session and simple_graph_execution_state (used by the distributed
runtime) to share the same graph execution and placement code, to reduce code
divergence.
Change: 122210923

1 files changed, 0 insertions, 2 deletions

diff --git a/tensorflow/core/common_runtime/simple_placer.cc b/tensorflow/core/common_runtime/simple_placer.cc
index 19a44c6a98..c14c37a4c3 100644
--- a/tensorflow/core/common_runtime/simple_placer.cc
+++ b/tensorflow/core/common_runtime/simple_placer.cc
@@ -469,7 +469,6 @@ class ColocationGraph {
 
       // If the NodeDef contains a device, then we interpret it as a
       // (partial) device specification.
-      string colocated_node_name;
       if (!node.def().device().empty()) {
         // The user has specified a device in the NodeDef, try to find a
         // valid device matching their specification in the set of
@@ -659,7 +658,6 @@ Status SimplePlacer::Run() {
     // and we can experiment with other algorithms when given a choice of
     // devices.
     node->set_assigned_device_name(devices[0]->name());
-
     // Log placement if log_device_placement is set.
     if (options_ && options_->config.log_device_placement()) {
       printf("%s: %s\n", node->name().c_str(),