Skip to content

Commit 1bdd349

Browse files
shadowdragon89tensorflow-copybara
authored andcommitted
Remove tensorflow/contrib/session_bundle as dependency for Tensorflow Serving.
PiperOrigin-RevId: 275281512
1 parent 1b33368 commit 1bdd349

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

55 files changed

+1724
-852
lines changed

tensorflow_serving/batching/BUILD

-1
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,6 @@ cc_test(
8585
"//tensorflow_serving/test_util",
8686
"@org_tensorflow//tensorflow/cc/saved_model:loader",
8787
"@org_tensorflow//tensorflow/cc/saved_model:tag_constants",
88-
"@org_tensorflow//tensorflow/contrib/session_bundle",
8988
"@org_tensorflow//tensorflow/core:core_cpu",
9089
"@org_tensorflow//tensorflow/core:framework",
9190
"@org_tensorflow//tensorflow/core:lib",

tensorflow_serving/batching/batching_session_test.cc

-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ limitations under the License.
1818
#include <gtest/gtest.h>
1919
#include "tensorflow/cc/saved_model/loader.h"
2020
#include "tensorflow/cc/saved_model/tag_constants.h"
21-
#include "tensorflow/contrib/session_bundle/session_bundle.h"
2221
#include "tensorflow/core/framework/tensor.h"
2322
#include "tensorflow/core/framework/tensor_shape.h"
2423
#include "tensorflow/core/framework/tensor_testutil.h"

tensorflow_serving/model_servers/BUILD

+4-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# Description: Model Server
22

3+
load("//tensorflow_serving:tensorflow_version.bzl", "if_not_v2", "if_v2")
4+
35
package(
46
default_visibility = [
57
"//tensorflow_serving:internal",
@@ -116,6 +118,7 @@ cc_test(
116118
"//tensorflow_serving/model_servers/test_util:storage_path_error_injecting_source_adapter",
117119
"//tensorflow_serving/model_servers/test_util:storage_path_error_injecting_source_adapter_proto",
118120
"//tensorflow_serving/test_util",
121+
"//tensorflow_serving/util:oss_or_google",
119122
"@com_google_protobuf//:cc_wkt_protos",
120123
"@org_tensorflow//tensorflow/core:lib",
121124
"@org_tensorflow//tensorflow/core:protos_all_cc",
@@ -162,7 +165,6 @@ cc_test(
162165
"//tensorflow_serving/test_util",
163166
"@org_tensorflow//tensorflow/cc/saved_model:loader",
164167
"@org_tensorflow//tensorflow/cc/saved_model:signature_constants",
165-
"@org_tensorflow//tensorflow/contrib/session_bundle",
166168
"@org_tensorflow//tensorflow/core:test",
167169
],
168170
)
@@ -291,8 +293,6 @@ cc_library(
291293
],
292294
)
293295

294-
load("//tensorflow_serving:tensorflow_version.bzl", "if_v2", "if_not_v2")
295-
296296
SUPPORTED_TENSORFLOW_OPS = if_v2([]) + if_not_v2([
297297
"@org_tensorflow//tensorflow/contrib:contrib_kernels",
298298
"@org_tensorflow//tensorflow/contrib:contrib_ops_op_lib",
@@ -412,6 +412,7 @@ py_test(
412412
deps = [
413413
"//tensorflow_serving/apis:model_service_proto_py_pb2",
414414
"//tensorflow_serving/apis:prediction_service_proto_py_pb2",
415+
"//tensorflow_serving/model_servers/test_util:tensorflow_model_server_test_base",
415416
"@org_tensorflow//tensorflow:tensorflow_py",
416417
"@six_archive//:six",
417418
],

tensorflow_serving/model_servers/get_model_status_impl_test.cc

-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ limitations under the License.
1919
#include <gtest/gtest.h>
2020
#include "tensorflow/cc/saved_model/loader.h"
2121
#include "tensorflow/cc/saved_model/signature_constants.h"
22-
#include "tensorflow/contrib/session_bundle/session_bundle.h"
2322
#include "tensorflow/core/lib/core/status_test_util.h"
2423
#include "tensorflow_serving/apis/model.pb.h"
2524
#include "tensorflow_serving/core/availability_preserving_policy.h"

tensorflow_serving/model_servers/server_core_test.cc

+7-1
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ limitations under the License.
3636
#include "tensorflow_serving/model_servers/test_util/storage_path_error_injecting_source_adapter.h"
3737
#include "tensorflow_serving/model_servers/test_util/storage_path_error_injecting_source_adapter.pb.h"
3838
#include "tensorflow_serving/test_util/test_util.h"
39+
#include "tensorflow_serving/util/oss_or_google.h"
3940

4041
namespace tensorflow {
4142
namespace serving {
@@ -879,7 +880,12 @@ TEST_P(ServerCoreTest, VersionLabelsNotAllowed) {
879880
INSTANTIATE_TEST_CASE_P(
880881
TestType, ServerCoreTest,
881882
::testing::Combine(
882-
::testing::Range(0, static_cast<int>(ServerCoreTest::NUM_TEST_TYPES)),
883+
IsTensorflowServingOSS()
884+
? ::testing::Range(
885+
static_cast<int>(test_util::ServerCoreTest::SAVED_MODEL),
886+
static_cast<int>(test_util::ServerCoreTest::SAVED_MODEL))
887+
: ::testing::Range(
888+
0, static_cast<int>(ServerCoreTest::NUM_TEST_TYPES)),
883889
::testing::Bool()));
884890

885891
INSTANTIATE_TEST_CASE_P(

0 commit comments

Comments
 (0)