Remove build warnings for graph part. (#1584)

Fix build warnings in graph. Signed-off-by: SighingSnow <1263750383@qq.com>
v6d-io · Sep 28, 2023 · d79f2e1 · d79f2e1
1 parent ff40aa2
commit d79f2e1
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 6 deletions.
diff --git a/modules/graph/fragment/arrow_fragment_impl.h b/modules/graph/fragment/arrow_fragment_impl.h
@@ -627,7 +627,6 @@ void ArrowFragment<OID_T, VID_T, VERTEX_MAP_T, COMPACT>::initDestFidList(
     for (auto e_label_id = 0; e_label_id < edge_label_num_; e_label_id++) {
       std::vector<int> id_num(ivnum_, 0);
       std::set<fid_t> dstset;
-      vertex_t v = *inner_vertices.begin();
 
       auto& fid_list = fid_lists[v_label_id][e_label_id];
       auto& fid_list_offset = fid_lists_offset[v_label_id][e_label_id];

diff --git a/modules/graph/loader/arrow_fragment_loader_impl.h b/modules/graph/loader/arrow_fragment_loader_impl.h
@@ -251,7 +251,6 @@ ArrowFragmentLoader<OID_T, VID_T>::addDataToExistedVLabel(
   std::shared_ptr<ArrowFragmentBase> frag;
 
   VY_OK_OR_RAISE(client_.GetObject(frag_id, frag));
-  const PropertyGraphSchema& schema = frag->schema();
 
   LOG_IF(INFO, !comm_spec_.worker_id()) << MARKER << "PROCESS-INPUTS-0";
 

diff --git a/modules/graph/tools/fragment_loader_impl.h b/modules/graph/tools/fragment_loader_impl.h
@@ -68,7 +68,7 @@ ObjectID load_graph(Client& client, grape::CommSpec& comm_spec,
     for (auto vfile : options.vfiles) {
       std::vector<std::string> sub_files;
       boost::split(sub_files, vfile, boost::is_any_of(";"));
-      for (int64_t i = 0; i < sub_files.size(); ++i) {
+      for (size_t i = 0; i < sub_files.size(); ++i) {
         loader_t adder(client, comm_spec, std::vector<std::string>{},
                        std::vector<std::string>{sub_files[i]}, options.directed,
                        options.generate_eid, options.retain_oid);
@@ -85,7 +85,7 @@ ObjectID load_graph(Client& client, grape::CommSpec& comm_spec,
     for (auto efile : options.efiles) {
       std::vector<std::string> sub_files;
       boost::split(sub_files, efile, boost::is_any_of(";"));
-      for (int64_t i = 0; i < sub_files.size(); ++i) {
+      for (size_t i = 0; i < sub_files.size(); ++i) {
         loader_t adder(client, comm_spec,
                        std::vector<std::string>{sub_files[i]},
                        std::vector<std::string>{}, options.directed,

diff --git a/modules/graph/vertex_map/arrow_vertex_map_impl.h b/modules/graph/vertex_map/arrow_vertex_map_impl.h
@@ -454,10 +454,10 @@ ObjectID ArrowVertexMap<OID_T, VID_T>::updateLabelVertexMap(
     for (int64_t i = 0; i < prev_array->length(); ++i) {
       prev_oids[prev_array->GetView(i)] = i;
     }
-    for (int i = 0; i < oid_arrays[fid].size(); ++i) {
+    for (size_t i = 0; i < oid_arrays[fid].size(); ++i) {
       for (int64_t j = 0; j < oid_arrays[fid][i]->length(); ++j) {
         if (prev_oids.find(oid_arrays[fid][i]->GetView(j)) == prev_oids.end()) {
-          builder.Append(oid_arrays[fid][i]->GetView(j));
+          RETURN_ON_ARROW_ERROR(builder.Append(oid_arrays[fid][i]->GetView(j)));
         }
       }
     }