Fix the wrong objectID in resolveVineyardObject.

modules/graph/loader/arrow_fragment_loader.cc

@@ -335,7 +335,7 @@
 
 boost::leaf::result<std::vector<std::vector<std::shared_ptr<arrow::Table>>>>
 GatherETables(Client& client,
               const std::vector<std::vector<ObjectID>>& estreams, int part_id,
               int part_num) {
   using batch_group_t = std::unordered_map<
       std::string, std::map<std::pair<std::string, std::string>,
@@ -344,9 +344,9 @@
   std::mutex mutex_for_results;
   auto reader = [&client, &mutex_for_results, &grouped_batches, part_id,
                  part_num](size_t const index,
                            ObjectID const estream) -> Status {
     std::vector<std::shared_ptr<arrow::RecordBatch>> batches;
     auto status = ReadRecordBatchesFromVineyard(client, estream, batches,
                                                 part_id, part_num);
     if (status.ok()) {
       std::lock_guard<std::mutex> scoped_lock(mutex_for_results);
@@ -366,28 +366,28 @@
             .emplace_back(batch);
       }
     } else {
       LOG(ERROR) << "Failed to read from stream " << ObjectIDToString(estream)
                  << ": " << status.ToString();
     }
     return Status::OK();
   };
 
   ThreadGroup tg;
   for (size_t index = 0; index < estreams.size(); ++index) {
     for (auto const& estream : estreams[index]) {
       tg.AddTask(reader, index, estream);
     }
   }
   tg.TakeResults();
 
   if (!estreams.empty() && grouped_batches.empty()) {
     grouped_batches[std::to_string(0)][std::make_pair("", "")] = {};
   }
 
   std::vector<std::vector<std::shared_ptr<arrow::Table>>> tables;
   for (auto const& group : grouped_batches) {
     std::shared_ptr<arrow::Table> table;
     std::vector<std::shared_ptr<arrow::Table>> subtables;
     for (auto const& subgroup : group.second) {
       if (subgroup.second.empty()) {
         table = nullptr;  // no tables at current worker
@@ -553,7 +553,7 @@
   // encoding: 'o' prefix for object id, and 's' prefix for object name.
   CHECK_OR_RAISE(!source.empty() && (source[0] == 'o' || source[0] == 's'));
   if (source[0] == 'o') {
-    sourceId = vineyard::ObjectIDFromString(source.substr(1));
+    sourceId = vineyard::ObjectIDFromString(source);
   } else {
     VY_OK_OR_RAISE(client_.GetName(source.substr(1), sourceId, true));
   }