Disable PopEF executable compression by default

AnthonyBarbier · georgepaw · commit 07ff34e69f6a · 2021-10-11T18:30:15.000+01:00
Reviewers: #tensorflow, #framework_ip_review_-_any_oss_or_third-party_code_use_has_been_approved, georgep Reviewed By: #tensorflow, #framework_ip_review_-_any_oss_or_third-party_code_use_has_been_approved, georgep Subscribers: georgep Maniphest Tasks: T46616 Differential Revision: https://phabricator.sourcevertex.net/D53191
diff --git a/tensorflow/compiler/plugin/poplar/driver/tools/poplar_executable_binary_file.cc b/tensorflow/compiler/plugin/poplar/driver/tools/poplar_executable_binary_file.cc
@@ -158,7 +158,7 @@ Status PoplarExecutableBinaryFile::Write(
   try {
     // Append the protobuf metadata.
     popef_opaque->stream << serialized;
-    popef_exe = popef_writer->createExecutable(exec_hash);
+    popef_exe = popef_writer->createExecutable(exec_hash, /*compress=*/false);
   } catch (const std::exception& e) {
     return PopEFExceptionToTensorflowStatus("[WriteOpaque]", e);
   }

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,7 @@ Status PoplarExecutableBinaryFile::Write(`
`158`	`158`	`try {`
`159`	`159`	`// Append the protobuf metadata.`
`160`	`160`	`popef_opaque->stream << serialized;`
`161`		`- popef_exe = popef_writer->createExecutable(exec_hash);`
	`161`	`+ popef_exe = popef_writer->createExecutable(exec_hash, /compress=/false);`
`162`	`162`	`} catch (const std::exception& e) {`
`163`	`163`	`return PopEFExceptionToTensorflowStatus("[WriteOpaque]", e);`
`164`	`164`	`}`