disable ncclAllReduceOp as well

helinwang · helinwang · commit c4720376c692 · 2018-04-02T11:33:02.000-07:00
diff --git a/paddle/fluid/operators/nccl_op_test.cu.cc b/paddle/fluid/operators/nccl_op_test.cu.cc
@@ -137,6 +137,8 @@ class NCCLTester : public ::testing::Test {
 TEST_F(NCCLTester, ncclInitOp) {}
 
 // ncclAllReduceOp with desc
+// TODO(helin): https://github.com/PaddlePaddle/Paddle/issues/9367
+/*
 TEST_F(NCCLTester, ncclAllReduceOp) {
   std::unique_ptr<f::OpDesc> op2(new f::OpDesc);
   op2->SetType("ncclAllReduce");
@@ -184,6 +186,7 @@ TEST_F(NCCLTester, ncclAllReduceOp) {
     }
   }
 }
+*/
 
 // ncclReduceOp with desc
 TEST_F(NCCLTester, ncclReduceOp) {

Original file line number	Diff line number	Diff line change
`@@ -137,6 +137,8 @@ class NCCLTester : public ::testing::Test {`
`137`	`137`	`TEST_F(NCCLTester, ncclInitOp) {}`
`138`	`138`
`139`	`139`	`// ncclAllReduceOp with desc`
	`140`	`+// TODO(helin): https://github.com/PaddlePaddle/Paddle/issues/9367`
	`141`	`+/*`
`140`	`142`	`TEST_F(NCCLTester, ncclAllReduceOp) {`
`141`	`143`	`std::unique_ptr<f::OpDesc> op2(new f::OpDesc);`
`142`	`144`	`op2->SetType("ncclAllReduce");`
`@@ -184,6 +186,7 @@ TEST_F(NCCLTester, ncclAllReduceOp) {`
`184`	`186`	`}`
`185`	`187`	`}`
`186`	`188`	`}`
	`189`	`+*/`
`187`	`190`
`188`	`191`	`// ncclReduceOp with desc`
`189`	`192`	`TEST_F(NCCLTester, ncclReduceOp) {`