openr/kvstore/KvStore.cpp (11 lines):
	- line 903: // TODO: evaluate if expired_key_vals num is in use
	- line 909: // TODO: evaludate if both keys are needed
	- line 1071: // TODO: migrate to secure thrift connection
	- line 1083: // TODO: leverage folly::Socket's KEEP_ALIVE option to manage this
	- line 1339: // TODO: now the key regex is hardcoded to match `adj:` key ONLY
	- line 2216: // TODO: remove this client call to use folly::Socket option to keep-alive
	- line 2242: // TODO: replace addPeers with addThriftPeers call
	- line 2440: // TODO: replace delPeers with delThriftPeers call
	- line 3061: // TODO: evaluate the condition later to add to pending collection
	- line 3389: // TODO: when native key subscription is supported. Handle callback here.
	- line 3605: * TODO: Ideally, make KvStore.h depend on KvStore-inl.h, which includes


openr/decision/SpfSolver.cpp (11 lines):
	- line 255: // TODO: These variables are deprecated and will go away soon
	- line 262: // TODO: With new PrefixMetrics we no longer treat routes differently based
	- line 281: // TODO: With new PrefixMetrics we no longer treat routes differently based
	- line 330: // TODO: We program self advertise prefix only iff, we're advertising our
	- line 363: //        - TODO: support this functionality for KSP2 forwarding algorithm
	- line 397: // TODO: bypass this code to allow UCMP paths between areas if
	- line 455: // SrPolicy TODO: (T94500292) before returning need to apply prepend label
	- line 791: // TODO: This is one off the hack to unblock special routing needs. With
	- line 1005: // TODO: This is one off the hack to unblock special routing needs. With
	- line 1163: // TODO Let's use strong-types for the bools to detect any abusement at the
	- line 1182: // TODO: Reorg this function to make the logic cleaner, and cleanup unused


openr/if/Types.thrift (6 lines):
	- line 151: * TODO: This is not yet supported and doesn't influence routing. Work is in
	- line 413: * TODO has: This is deprecated. Instead use `metrics` field, it is compact
	- line 549: * @deprecated TODO - This should be removed in favor of perfEvents in
	- line 602: * @deprecated TODO - This field is of no relevance
	- line 679: * TODO: Move this to Types.cpp
	- line 696: * TODO: Move this to Types.cpp


openr/link-monitor/LinkMonitor.cpp (6 lines):
	- line 132: // TODO: remove this with strict Open/R initialization sequence
	- line 235: // TODO: Handle InitializationEvent
	- line 254: // TODO: remove this queue to reduce KvStore dependency
	- line 350: // TODO: address value of peerAddr under system test environment
	- line 364: // TODO: remove `isRestarting` flag once enable_ordered_adj_publication is
	- line 926: // TODO: once `enable_ordered_adj_publication` is enabled everywhere, the


openr/prefix-manager/PrefixManager.cpp (5 lines):
	- line 90: // TODO: deprecate this after prefix_hold_time_s after OpenR initialization
	- line 770: // TODO: report per-area advertised prefixes if openr is running in
	- line 1546: // TODO: Consider moving static route generation and best entry selection
	- line 1578: // TODO: treat bgprib as another area, move this logic into syncKvStore()
	- line 1732: // TODO: remove this when advertise RibUnicastEntry for routes to delete


openr/decision/SpfSolver.h (3 lines):
	- line 31: // TODO: Remove once we move to metrics selection
	- line 239: * TODO: After selectBestRoutes() is cleaned up, merge selectBestRoutes() and
	- line 254: // TODO: This should go away, once Open/R policy is in place. The overloaded


openr/config/Config.cpp (3 lines):
	- line 205: // TODO remove once transition to areas is complete
	- line 543: // Hack for transioning phase. TODO: Remove after coop is on-boarded
	- line 685: * TODO: This is the util method to do a translation from:


build/fbcode_builder/CMake/FBPythonBinary.cmake (3 lines):
	- line 50: # TODO: On Windows we require the Python libraries as well.
	- line 187: # TODO: generate a main executable that will invoke Python3
	- line 420: # TODO: In the future it would be nice to support pre-compiling the source


openr/decision/RibEntry.h (3 lines):
	- line 24: // TODO: should this be map<area, nexthops>?
	- line 91: // TODO: rename this func
	- line 102: // TODO: rename this func


openr/common/LsdbUtil.cpp (3 lines):
	- line 515: // TODO: refactor this as prefixEntry struct has been
	- line 517: // TODO: Create and return thrift::PrefixEntry as a
	- line 543: // TODO: Audit which util functions in this file are only used in unit test.


openr/watchdog/Watchdog.cpp (3 lines):
	- line 170: // TODO: T98478475 - Currently we only log the counters but don't take any
	- line 183: // TODO: 1. Handle the delete scenario where a reader disconnects
	- line 184: // TODO: 2. Assign keys to each reader so we can record the stats without


openr/common/LsdbTypes.h (2 lines):
	- line 149: * TODO: Deprecate NeighborEvents and use NeighborEvent in NeighborInitEvent
	- line 384: // TODO: thrift::InterfaceInfo to be deprecated


openr/kvstore/KvStoreWrapper.cpp (2 lines):
	- line 250: // TODO: add timeout to avoid infinite waiting
	- line 268: // TODO: add timeout to avoid infinite waiting


openr/common/Types.h (2 lines):
	- line 22: // TODO: remove KvStore URLs
	- line 232: * TODO: remove this once openr_intialization is by default enabled


build/fbcode_builder/getdeps.py (2 lines):
	- line 834: # TODO: we could just go ahead and build it here, but I
	- line 904: # TODO: Break up complex function


build/fbcode_builder/getdeps/platform.py (2 lines):
	- line 56: # TODO: Ideally, this function would inspect the current cgroup for any
	- line 88: # TODO: There may be some way to approximate an availability


openr/decision/RouteUpdate.h (2 lines):
	- line 94: // TODO: rename this func
	- line 116: // TODO: rename this func


openr/decision/Decision.cpp (2 lines):
	- line 737: // TODO: deprecate area_ref()
	- line 794: // TODO: avoid decoding from string by injecting data-structures


build/fbcode_builder/CMake/FBCMakeParseArgs.cmake (2 lines):
	- line 49: # TODO: We aren't extremely strict about error checking for one-value
	- line 125: # TODO: Our caller still won't be able to distinguish between an empty


build/fbcode_builder/getdeps/buildopts.py (1 line):
	- line 521: # TODO: `mkscratch` doesn't currently know how best to place things on


openr/spark/Spark.h (1 line):
	- line 188: // TODO: standardize Spark inline documentation


openr/py/openr/cli/utils/utils.py (1 line):
	- line 1612: # TODO remove hard dependency on json output format in fbossdeploy/fcr


build/fbcode_builder/docker_builder.py (1 line):
	- line 37: # system packages are installed.  TODO: For users not defined in the


openr/kvstore/KvStoreUtil.cpp (1 line):
	- line 229: // TODO: `ttlVersion` and `ttl` value can be different on neighbor nodes.


openr/common/Util.h (1 line):
	- line 68: * TODO: Remove the API in favor of other one


openr/if/Network.thrift (1 line):
	- line 73: // TODO: Define internal representation of NextHop within Open/R. We shouldn't


openr/link-monitor/LinkMonitor.h (1 line):
	- line 43: // TODO: remove `isRestarting` flag once enable_ordered_adj_publication is


build/fbcode_builder/getdeps/dyndeps.py (1 line):
	- line 117: # TODO: audit that both source paths have the same inode number


openr/if/KvStore.thrift (1 line):
	- line 537: * TODO: Expose more detailed information `expiredKeyVals` so that subscribers


openr/if/OpenrConfig.thrift (1 line):
	- line 848: * TODO: It is temporary & will go away once new prefix metrics is rolled out


openr/py/openr/cli/commands/kvstore.py (1 line):
	- line 49: # TODO: remove self.area_feature as it will be supported by default


openr/messaging/Queue.h (1 line):
	- line 31: std::string queueId; // TODO: Change to const post T98477650


openr/prefix-manager/PrefixManager.h (1 line):
	- line 404: // TODO: Merge this with advertiseStatus_.


openr/py/openr/cli/clis/decision.py (1 line):
	- line 124: TODO: Fix json to be combined for all areas ...


openr/platform/NetlinkFibHandler.h (1 line):
	- line 108: * TODO: Migrate BGP++ to stream API for neighbor notifications. Also need to


openr/common/LsdbUtil.h (1 line):
	- line 340: * TODO: Deprecated and the support for best route selection based on metric


openr/decision/PrefixState.h (1 line):
	- line 54: // TODO: Also maintain clean list of reachable prefix entries. A node might


openr/messaging/ReplicateQueue-inl.h (1 line):
	- line 127: // TODO T98477650 : We need to maintain proper queueIds instead of


build/fbcode_builder/getdeps/builder.py (1 line):
	- line 122: # TODO: Introduce some sort of defaults config that can select by


openr/spark/Spark.cpp (1 line):
	- line 1416: // TODO: Spark is yet to support area change due to dynamic configuration.


build/fbcode_builder/getdeps/copytree.py (1 line):
	- line 75: TODO: I'd like to either extend this or add a variant that


openr/allocators/RangeAllocator.h (1 line):
	- line 129: // TODO: remove the raw ptr usage of KvStore.


openr/Main.cpp (1 line):
	- line 94: // TODO: handle case when openr received SIGTERM when waiting for fibService


openr/py/openr/cli/commands/decision.py (1 line):
	- line 138: # TODO: we should ideally get address with last bit


build/fbcode_builder/getdeps/py_wheel_builder.py (1 line):
	- line 158: # TODO: process the subdirectories of data_dir_name