spark-load/spark-load-common/src/main/java/org/apache/doris/common/io/Roaring64Map.java (12 lines): - line 74: // TODO: I would prefer not managing arrays myself - line 380: // TODO .size is too slow - line 423: // TODO: we should handle cardinality fitting in an unsigned int - line 657: // TODO .size on tailMap make an iterator: arg - line 660: // TODO: It should be possible to compute indexOk based on sortedHighs array - line 746: // TODO softer growing factor - line 775: // TODO: The entry comes from a NavigableMap: it may be quite cheap to know the next high - line 801: // TODO Reviewers: is it a good idea to rely on BitmapDataProvider except in methods - line 803: // TODO This code may lead to closing a buffer Bitmap in current Navigable even if current is - line 855: // TODO Reviewers: is it a good idea to rely on BitmapDataProvider except in methods - line 857: // TODO This code may lead to closing a buffer Bitmap in current Navigable even if current is - line 1339: // TODO Is it relevant to calling .flip with a cast? spark-load/spark-load-dpp/src/main/java/org/apache/doris/load/loadv2/dpp/DorisRangePartitioner.java (1 line): - line 59: // TODO: optimize this by use binary search