(incubator-graphar) branch main updated: fix(c++): Fix debug building warning (#504)

[weibin]

This is an automated email from the ASF dual-hosted git repository.

weibin pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-graphar.git


The following commit(s) were added to refs/heads/main by this push:
     new cbba4f5  fix(c++): Fix debug building warning (#504)
cbba4f5 is described below

commit cbba4f5e34586f68963086fbd45e04968e8844e0
Author: Weibin Zeng <qiaozi....@alibaba-inc.com>
AuthorDate: Fri May 31 09:57:11 2024 +0800

    fix(c++): Fix debug building warning (#504)
---
 cpp/benchmarks/arrow_chunk_reader_benchmark.cc | 24 ++++++++++++------------
 cpp/examples/high_level_reader_example.cc      |  2 +-
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/cpp/benchmarks/arrow_chunk_reader_benchmark.cc 
b/cpp/benchmarks/arrow_chunk_reader_benchmark.cc
index 9f761d7..db241ef 100644
--- a/cpp/benchmarks/arrow_chunk_reader_benchmark.cc
+++ b/cpp/benchmarks/arrow_chunk_reader_benchmark.cc
@@ -87,9 +87,9 @@ BENCHMARK_DEFINE_F(BenchmarkFixture, 
VertexPropertyArrowChunkReaderReadChunk)
   }
   auto reader = maybe_reader.value();
   for (auto _ : state) {
-    reader->seek(0);
-    reader->GetChunk();
-    reader->next_chunk();
+    assert(reader->seek(0).ok());
+    assert(reader->GetChunk().status().ok());
+    assert(reader->next_chunk().ok());
   }
 }
 
@@ -103,9 +103,9 @@ BENCHMARK_DEFINE_F(BenchmarkFixture, 
AdjListArrowChunkReaderReadChunk)
   }
   auto reader = maybe_reader.value();
   for (auto _ : state) {
-    reader->seek(0);
-    reader->GetChunk();
-    reader->next_chunk();
+    assert(reader->seek(0).ok());
+    assert(reader->GetChunk().status().ok());
+    assert(reader->next_chunk().ok());
   }
 }
 
@@ -119,9 +119,9 @@ BENCHMARK_DEFINE_F(BenchmarkFixture, 
AdjListOffsetArrowChunkReaderReadChunk)
   }
   auto reader = maybe_reader.value();
   for (auto _ : state) {
-    reader->seek(0);
-    reader->GetChunk();
-    reader->next_chunk();
+    assert(reader->seek(0).ok());
+    assert(reader->GetChunk().status().ok());
+    assert(reader->next_chunk().ok());
   }
 }
 
@@ -136,9 +136,9 @@ BENCHMARK_DEFINE_F(BenchmarkFixture, 
AdjListPropertyArrowChunkReaderReadChunk)
   }
   auto reader = maybe_reader.value();
   for (auto _ : state) {
-    reader->seek(0);
-    reader->GetChunk();
-    reader->next_chunk();
+    assert(reader->seek(0).ok());
+    assert(reader->GetChunk().status().ok());
+    assert(reader->next_chunk().ok());
   }
 }
 
diff --git a/cpp/examples/high_level_reader_example.cc 
b/cpp/examples/high_level_reader_example.cc
index 1728329..3725618 100644
--- a/cpp/examples/high_level_reader_example.cc
+++ b/cpp/examples/high_level_reader_example.cc
@@ -35,7 +35,7 @@ void vertices_collection(
   auto vertices = maybe_vertices_collection.value();
 
   // use vertices collection
-  auto count = 0;
+  size_t count = 0;
   // iterate through vertices collection
   for (auto it = vertices->begin(); it != vertices->end(); ++it) {
     count++;


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@graphar.apache.org
For additional commands, e-mail: commits-h...@graphar.apache.org