Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22819 )

Change subject: [util] introduce ChunkedBitmapIterator
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22819/4/src/kudu/util/bitmap.h
File src/kudu/util/bitmap.h:

http://gerrit.cloudera.org:8080/#/c/22819/4/src/kudu/util/bitmap.h@147
PS4, Line 147:   virtual size_t Next(bool* value) = 0;
> That's a good question.
I came up with an approach that doesn't require using ChunkedBitmapIterator: 
it's enough to have just regular BitmapIterator, so this is no longer an issue.



--
To view, visit http://gerrit.cloudera.org:8080/22819
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I143e3caa7cd8d48be8034c630f79562c2fc1282a
Gerrit-Change-Number: 22819
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
Gerrit-Comment-Date: Sat, 30 Aug 2025 05:16:44 +0000
Gerrit-HasComments: Yes

Reply via email to