fix: make is_bulk_loading filed optional in query_bulk_load_response (#…

…1009)
apache · Jan 6, 2022 · f8d632f · f8d632f
1 parent 6af2c32
commit f8d632f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/common/bulk_load.thrift b/src/common/bulk_load.thrift
@@ -213,5 +213,5 @@ struct query_bulk_load_response
     // detailed bulk load state for each replica
     6:list<map<dsn.rpc_address, partition_bulk_load_state>> bulk_load_states;
     7:optional string                                       hint_msg;
-    8:bool                                                  is_bulk_loading;
+    8:optional bool                                         is_bulk_loading;
 }
diff --git a/src/meta/meta_bulk_load_service.cpp b/src/meta/meta_bulk_load_service.cpp
@@ -1505,7 +1505,7 @@ void bulk_load_service::on_query_bulk_load_status(query_bulk_load_rpc rpc)
         }
     }
 
-    response.is_bulk_loading = app->is_bulk_loading;
+    response.__set_is_bulk_loading(app->is_bulk_loading);
 
     if (!app->is_bulk_loading && bulk_load_status::BLS_FAILED == response.app_status) {
         response.err = get_app_bulk_load_err_unlocked(app_id);