Summary: 99 instances, 91 unique

Text	Count
// TODO: Audit which util functions in this file are only used in unit test.	1
// TODO: refactor this as prefixEntry struct has been	1
TODO: I'd like to either extend this or add a variant that	1
// TODO: Define internal representation of NextHop within Open/R. We shouldn't	1
* TODO: This is not yet supported and doesn't influence routing. Work is in	1
// TODO T98477650 : We need to maintain proper queueIds instead of	1
// TODO: avoid decoding from string by injecting data-structures	1
// TODO: report per-area advertised prefixes if openr is running in	1
// TODO: We program self advertise prefix only iff, we're advertising our	1
// TODO: thrift::InterfaceInfo to be deprecated	1
// TODO: standardize Spark inline documentation	1
* TODO: remove this once openr_intialization is by default enabled	1
// TODO: migrate to secure thrift connection	1
* TODO: Deprecate NeighborEvents and use NeighborEvent in NeighborInitEvent	1
# TODO: Break up complex function	1
* TODO: Remove the API in favor of other one	1
// TODO Let's use strong-types for the bools to detect any abusement at the	1
// TODO: remove this with strict Open/R initialization sequence	1
// TODO: 1. Handle the delete scenario where a reader disconnects	1
// TODO: evaludate if both keys are needed	1
# TODO: audit that both source paths have the same inode number	1
* TODO: It is temporary & will go away once new prefix metrics is rolled out	1
// Hack for transioning phase. TODO: Remove after coop is on-boarded	1
// TODO: Spark is yet to support area change due to dynamic configuration.	1
# TODO: process the subdirectories of data_dir_name	1
# TODO: Introduce some sort of defaults config that can select by	1
// SrPolicy TODO: (T94500292) before returning need to apply prepend label	1
* TODO: Ideally, make KvStore.h depend on KvStore-inl.h, which includes	1
# TODO: In the future it would be nice to support pre-compiling the source	1
// TODO: This is one off the hack to unblock special routing needs. With	2
// TODO: Reorg this function to make the logic cleaner, and cleanup unused	1
// TODO: once `enable_ordered_adj_publication` is enabled everywhere, the	1
// TODO: add timeout to avoid infinite waiting	2
std::string queueId; // TODO: Change to const post T98477650	1
// TODO: Merge this with advertiseStatus_.	1
* TODO: Deprecated and the support for best route selection based on metric	1
// TODO: evaluate the condition later to add to pending collection	1
* TODO: After selectBestRoutes() is cleaned up, merge selectBestRoutes() and	1
// TODO: evaluate if expired_key_vals num is in use	1
// TODO: replace delPeers with delThriftPeers call	1
# TODO: On Windows we require the Python libraries as well.	1
// TODO: These variables are deprecated and will go away soon	1
# TODO: Our caller still won't be able to distinguish between an empty	1
// TODO: deprecate this after prefix_hold_time_s after OpenR initialization	1
* TODO has: This is deprecated. Instead use `metrics` field, it is compact	1
# TODO: we could just go ahead and build it here, but I	1
// TODO: should this be map<area, nexthops>?	1
// TODO: Create and return thrift::PrefixEntry as a	1
// TODO: Consider moving static route generation and best entry selection	1
# TODO: We aren't extremely strict about error checking for one-value	1
// TODO: With new PrefixMetrics we no longer treat routes differently based	2
// TODO: handle case when openr received SIGTERM when waiting for fibService	1
// TODO: remove this client call to use folly::Socket option to keep-alive	1
//        - TODO: support this functionality for KSP2 forwarding algorithm	1
// TODO: leverage folly::Socket's KEEP_ALIVE option to manage this	1
// TODO: treat bgprib as another area, move this logic into syncKvStore()	1
// TODO: remove this queue to reduce KvStore dependency	1
# TODO: `mkscratch` doesn't currently know how best to place things on	1
// TODO: when native key subscription is supported. Handle callback here.	1
// TODO: This should go away, once Open/R policy is in place. The overloaded	1
// TODO: address value of peerAddr under system test environment	1
// TODO: 2. Assign keys to each reader so we can record the stats without	1
// TODO: remove this when advertise RibUnicastEntry for routes to delete	1
// TODO: remove `isRestarting` flag once enable_ordered_adj_publication is	2
// TODO: deprecate area_ref()	1
# TODO: we should ideally get address with last bit	1
// TODO: Also maintain clean list of reachable prefix entries. A node might	1
// TODO: rename this func	4
* TODO: Expose more detailed information `expiredKeyVals` so that subscribers	1
# system packages are installed.  TODO: For users not defined in the	1
* TODO: Migrate BGP++ to stream API for neighbor notifications. Also need to	1
// TODO: T98478475 - Currently we only log the counters but don't take any	1
* TODO: This is the util method to do a translation from:	1
// TODO: now the key regex is hardcoded to match `adj:` key ONLY	1
* @deprecated TODO - This should be removed in favor of perfEvents in	1
TODO: Fix json to be combined for all areas ...	1
// TODO: bypass this code to allow UCMP paths between areas if	1
// TODO: remove KvStore URLs	1
// TODO remove once transition to areas is complete	1
# TODO: generate a main executable that will invoke Python3	1
# TODO: There may be some way to approximate an availability	1
# TODO: remove self.area_feature as it will be supported by default	1
// TODO: Remove once we move to metrics selection	1
// TODO: `ttlVersion` and `ttl` value can be different on neighbor nodes.	1
# TODO: Ideally, this function would inspect the current cgroup for any	1
// TODO: remove the raw ptr usage of KvStore.	1
* @deprecated TODO - This field is of no relevance	1
// TODO: Handle InitializationEvent	1
# TODO remove hard dependency on json output format in fbossdeploy/fcr	1
* TODO: Move this to Types.cpp	2
// TODO: replace addPeers with addThriftPeers call	1