TODOs cleanup

Summary: Clean up out-of-date TODOs; Mark the task to still valid TODO.

Reviewed By: haowu14

Differential Revision: D55090262

fbshipit-source-id: cbc9d7bc041b12c0378f92c13c8e5115e869aa1c

Author: Jiayue Bao

cachelib/allocator/ChainedHashTable.h

@@ -559,7 +559,6 @@ class ChainedHashTable {
         return !(*this == other);
       }
 
-      // TODO(jiayueb): change to return ReadHandle after fixing all the breaks
       const Handle& asHandle() { return curr(); }
 
       // reset the Iterator to begin of container

cachelib/navy/common/Device.cpp

@@ -1129,7 +1129,7 @@ folly::File openCacheFile(const std::string& fileName,
   }
 
 #ifndef MISSING_FALLOCATE
-  // TODO(jiayueb): make allocate flag user configurable and migrate the
+  // TODO T182882306: make allocate flag user configurable and migrate the
   // existing use cases
   // fallocate the file if requesting a larger file size and allocate flag is
   // set

cachelib/rust/src/cachelib.cpp

@@ -179,7 +179,6 @@ std::unique_ptr<LruItemHandle> find_item(const LruAllocator& cache,
                                          folly::StringPiece key) {
   auto item = const_cast<LruAllocator&>(cache).find(key);
   if (item) {
-    // TODO(jiayueb) remove toWriteHandle() after finishing R/W handle migration
     return std::make_unique<LruItemHandle>(std::move(item).toWriteHandle());
   } else {
     return std::unique_ptr<LruItemHandle>();