diff --git a/paddle/fluid/framework/selected_rows.cc b/paddle/fluid/framework/selected_rows.cc
index 578740ab20f3c1cdfe0b41e77102f08e310fd348..c41b95193e21e7c16b80623f0290e7ff737bb22e 100644
--- a/paddle/fluid/framework/selected_rows.cc
+++ b/paddle/fluid/framework/selected_rows.cc
@@ -70,7 +70,7 @@ struct TensorFillVisitor {
 
   template <typename T>
   void apply() const {
-    // TODO(Yancey1989): support other place
+    // TODO(qiao): support other place
     platform::CPUPlace cpu;
     auto* tensor_data = dst_->mutable_data<T>(cpu);
     auto* start = tensor_data + dst_offset_;
diff --git a/paddle/fluid/operators/lookup_sparse_table_op.cc b/paddle/fluid/operators/lookup_sparse_table_op.cc
index 01d67990a87a0db7058cf3b24182a8c003e1bb1c..a6843f20a59a23bd4e875b0f96524cc8d7aa46d6 100644
--- a/paddle/fluid/operators/lookup_sparse_table_op.cc
+++ b/paddle/fluid/operators/lookup_sparse_table_op.cc
@@ -94,7 +94,7 @@ class LookupSparseTableOpMaker : public framework::OpProtoAndCheckerMaker {
         .SetDefault(true);
     AddAttr<bool>("is_test",
                   "In test mode, lookup_sparse_table will "
-                  "return a default value for unknown id")
+                  "return a 0 for unknown id")
         .SetDefault(false);
     AddComment(R"DOC(
 Lookup Sprase Tablel Operator.