velox/dwio/dwrf/writer/ColumnWriter.cpp (18 lines):
	- line 217: // TODO: passing factory method into the dict encoder also works
	- line 269: // TODO: It would be nice if the suppression could be pushed down to
	- line 286: // FIXME: call base class set encoding first to deal with sequence and
	- line 304: // TODO: the only way useDictionaryEncoding_ right now is
	- line 334: // TODO: hopefully not required in the future, but need to solve the problem
	- line 347: // TODO: passing factory method into the dict encoder also works
	- line 404: // TODO: wonder if this should be final dictionary size instead. In that
	- line 554: // TODO: T45220726 Reduce memory usage in this method by
	- line 578: // TODO: T45260340 Unfortunately standard usage of the index
	- line 849: // TODO: T46365785 It would be nice if the suppression
	- line 873: // FIXME: call base class set encoding first to deal with sequence and
	- line 891: // TODO: the only way useDictionaryEncoding_ right now is
	- line 926: // TODO: hopefully not required in the future, but need to solve the problem
	- line 1138: // TODO Optimize to avoid copy
	- line 1179: // TODO: T45220726 Reduce memory usage in this method by
	- line 1216: // TODO: T45260340 Unfortunately standard usage of the index
	- line 1241: // TODO: optimize this branching either through restoring visitor
	- line 1487: // TODO Optimize to avoid copy


velox/experimental/codegen/benchmark/StringBenchmarks.cpp (8 lines):
	- line 158: // TODO: enable generating strings that represent numbers
	- line 175: // TODO: Casting fails here because str1 is not a valid double string.
	- line 199: // TODO: This test is failling with a exceptioon  because str1 (and possibly
	- line 255: // TODO: investigate why vectorFunc is much faster than compiled
	- line 274: // TODO: This basically here to track the benefit of constructing constant
	- line 397: // TODO 1: Investigate copying
	- line 398: // TODO 2: Investigate using a global bump allocator instead of std::string
	- line 399: // TODO 3: Add ascii propagation optimization


velox/experimental/codegen/vector_function/VectorReader-inl.h (7 lines):
	- line 47: // TODO: move readers to different directory
	- line 86: // TODO when read only vector does not have nulls we dont need to
	- line 103: // FIXME: generated code should avoid calling on this on the writer and
	- line 186: // TODO when read only vector does not have nulls we dont need to allocate
	- line 291: // TODO: avoid copying constant string by allocating them on Velox buffer
	- line 292: // TODO: specialize for default nulls
	- line 310: // TODO when read only vector does not have nulls we dont need to allocate


velox/substrait/SubstraitToVeloxPlan.cpp (7 lines):
	- line 206: // TODO: Currently Filter is skipped because Filter is Pushdowned to
	- line 309: // TODO: Use the names as the output names for the whole computing.
	- line 354: // TODO: Support different types here.
	- line 422: // TODO: Add different types' support here.
	- line 429: // TODO: Only direct reference is considered here.
	- line 436: // TODO: Only double is considered here.
	- line 506: // TODO: Only and relation is supported here.


velox/expression/VectorUdfTypeSystem.h (7 lines):
	- line 547: // TODO: deprecate this once all proxies introduced.
	- line 1133: // TODO: This is temporary until the new map writer interface is completed.
	- line 1483: // TODO (kevinwilfong): Add support for Generics in callNullFree.
	- line 1492: // TODO (kevinwilfong): Add support for Generics in callNullFree.
	- line 1499: // TODO (kevinwilfong): Add support for Generics in callNullFree.
	- line 1506: // TODO (kevinwilfong): Add support for Generics in callNullFree.
	- line 1518: // TODO: This is temporary until the new map writer interface is completed.


velox/common/memory/Memory.h (6 lines):
	- line 110: // TODO: might be able to move implementation to abstract class
	- line 269: // TODO: move to factory pattern with type trait.
	- line 274: // TODO: might be able to collapse this with templated class
	- line 376: // TODO: Consider putting these in base class also.
	- line 457: // TODO: consider returning bool instead.
	- line 486: // TODO: perhaps the root pool should be a specialized pool that


velox/dwio/dwrf/reader/ColumnReader.cpp (6 lines):
	- line 1167: // TODO Reuse memory
	- line 1188: BufferPtr(nullptr), // TODO nulls
	- line 1197: // TODO Reuse memory
	- line 1210: BufferPtr(nullptr), // TODO nulls
	- line 1470: // TODO Reuse memory
	- line 1479: // TODO Reuse memory


velox/experimental/codegen/transform/utils/adapters.h (5 lines):
	- line 31: // TODO:  this assume that nodePtr is never null, we need to add a constraints
	- line 63: // TODO:  this assume that nodePtr is never null, we need to add a constraints
	- line 76: // TODO:  this assume that nodePtr is never null, we need to add a constraints
	- line 99: * Constraints (TODO : define formaly those constraints in term of c++ concepts)
	- line 152: // TODO: probably can introduce a macro here.


velox/core/Context.h (5 lines):
	- line 161: // TODO (bharathb) : Either disallow or flatten config stacks here.
	- line 164: // TODO (bharathb) : this need to be atomic.
	- line 177: // TODO (bharathb) : replace dynamic_pointer_cast with
	- line 272: // TODO (bharathb): Look for a lock free datastructure(replace with a list).
	- line 273: // TODO (bharathb) : We notify everyone irrespective of whether their


velox/experimental/codegen/code_generator/ExprCodeGenerator.cpp (5 lines):
	- line 28: // TODO add codgenType inside velox type traits for now we are using
	- line 68: /// TODO extract this onto a separate process
	- line 124: // TODO: support null on failure
	- line 396: // TODO: Rename this to something more descriptive
	- line 416: // TODO: let's templatise the invoke function;


velox/experimental/codegen/transform/utils/utils.h (5 lines):
	- line 47: * TODO: Introduce more refined of the tree concept : mutable tree
	- line 62: // TODO: relax this constraint to allow more pointer type
	- line 72: * TODO: Make sure that this class respects the range concept
	- line 73: * TODO: Implement a lazy version
	- line 128: * @tparam NodeTransformer  (TODO: see NodeTransformer concept)


velox/experimental/codegen/ast/ASTNode.cpp (4 lines):
	- line 100: // TODO: support eager format
	- line 126: // TODO: double check if we need to check on eagerness here.
	- line 210: // FIXME: Unless query optimizer took care of this already, othersie this
	- line 272: // TODO: this is a work around for UDFs that do not respect the templated


velox/type/Type.h (4 lines):
	- line 130: // TODO Implement
	- line 523: // TODO: velox throws here for non fixed width types.
	- line 533: // TODO: velox implementation is in cpp
	- line 1481: // TODO: maybe do something smarter than just matching any shared_ptr, e.g. we


velox/experimental/codegen/CodegenCompiledExpressionTransform.h (4 lines):
	- line 138: // TODO: Move this into a file
	- line 181: /// TODO: use a range_view instead of a containers as arg
	- line 206: // TODO: We probably need here to have a better ordering of the columns
	- line 525: // TODO: We should exclude here direct access columns.


velox/dwio/dwrf/writer/WriterShared.cpp (4 lines):
	- line 203: // TODO: override getCurrentBytes() to be lock-free for leaf level pools.
	- line 277: // TODO: extract context.canSwitchEncoding().
	- line 357: // TODO: T45025996 Discard all empty streams at flush time.
	- line 432: // TODO: what does this try to capture?


velox/experimental/codegen/vector_function/StringTypes.h (4 lines):
	- line 56: // TODO we can allocate on the pool
	- line 181: // TODO: avoid this copy when oldBuffer and newBuffer are sequential in
	- line 281: // TODO:
	- line 293: // TODO:// coming have some thoughts


velox/experimental/codegen/vector_function/GeneratedVectorFunction-inl.h (4 lines):
	- line 187: // TODO: We should probably move this loop inside ensureWritable
	- line 217: // TODO: write now we are sharing everything, this not ideal. We should do
	- line 387: // TODO: Missing implementation;
	- line 449: /// TODO:  This might need to be improved in case func return ref types.


velox/dwio/dwrf/reader/SelectiveStringDictionaryColumnReader.cpp (4 lines):
	- line 162: // TODO Reuse memory
	- line 183: BufferPtr(nullptr), // TODO nulls
	- line 188: // TODO Reuse memory
	- line 201: BufferPtr(nullptr), // TODO nulls


velox/connectors/hive/HiveConnector.cpp (3 lines):
	- line 84: // TODO: Wire up serde properties to writer configs.
	- line 172: // TODO Remove this check when Presto is fixed to not specify a filter
	- line 524: // TODO Check if remaining filter has a conjunct that doesn't depend on any


velox/exec/MergeJoin.cpp (3 lines):
	- line 236: // TODO Re-use output_ columns when possible.
	- line 271: // TODO Refactor this logic into a method on BaseVector.
	- line 362: // TODO Finish early if ran out of data on either side of the join.


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


velox/experimental/codegen/library_loader/FunctionTable.h (3 lines):
	- line 69: // TODO: Investigate why std::shared_ptr here is seen as incomplete type
	- line 70: // TODO: Is it safe to have std::shared_ptr cross ABI boundaries?
	- line 77: // TODO: Add interface to save/Restor the state of an expression


velox/dwio/dwrf/common/RLEv2.cpp (3 lines):
	- line 393: // TODO: something more efficient than resize
	- line 400: // TODO: something more efficient than resize
	- line 403: // TODO: Skip corrupt?


velox/connectors/hive/storage_adapters/s3fs/S3FileSystem.cpp (3 lines):
	- line 114: // TODO: allocate from a memory pool
	- line 132: // TODO: Check if any buffers are being used by the S3 library
	- line 369: // TODO: Support multiple S3FileSystem instances using a cache


velox/connectors/Connector.h (3 lines):
	- line 85: // TODO maybe at some point we want to make it async
	- line 132: // TODO Allow DataSource to indicate that it is blocked (say waiting for IO)
	- line 222: // TODO Generalize to specify TableHandle/Layout and ColumnHandles.


velox/type/Timestamp.h (2 lines):
	- line 122: return StringView("TODO: Implement");
	- line 168: // TODO Implement


velox/vector/BaseVector.cpp (2 lines):
	- line 486: // TODO Figure out how to allow memory reuse for a newly loaded vector.
	- line 596: // TODO: Reuse the indices if 'source' is already a dictionary and


velox/vector/arrow/Bridge.cpp (2 lines):
	- line 298: // TODO: need to figure out how we'll map this since in Velox we currently
	- line 372: // TODO: No dictionaries for now.


velox/experimental/codegen/code_generator/ExprCodeGenerator.h (2 lines):
	- line 32: /// TODO: Update this to reflected latest changes in D26383419
	- line 123: /// TODO: We should really be story full library information here instead


velox/dwio/dwrf/common/CachedBufferedInput.h (2 lines):
	- line 33: // like pins into file handle caches. TODO: Make file handle cache
	- line 45: // unique_ptr. TODO: Make all streams owned via shared_ptr.


velox/experimental/codegen/external_process/subprocess.h (2 lines):
	- line 85: // TODO: the on_exit handler is not triggered
	- line 95: // TODO: Normally, we would want to handle this in the on_error callback.


velox/substrait/proto/substrait/algebra.proto (2 lines):
	- line 40: // TODO: nodes, cpu threads/%, memory, iops, etc.
	- line 630: // TODO: should allow expressions


velox/functions/sparksql/String.h (2 lines):
	- line 104: // TODO: Once C++20 supported we may want to replace this with
	- line 130: // TODO Once C++20 supported we may want to replace this with


velox/functions/lib/DateTimeFormatter.cpp (2 lines):
	- line 281: // TODO: implement short name time zone, need a map from full name to
	- line 293: // TODO: implement timezone offset id formatting, need a map from full


velox/functions/sparksql/SplitFunctions.cpp (2 lines):
	- line 117: // TODO: Add support for zero-length pattern, 2-character pattern
	- line 118: // TODO: add support for general regex pattern using R2


velox/experimental/codegen/external_process/Command.h (2 lines):
	- line 29: /// TODO 1: add reference to the working directory, and the search path.
	- line 48: /// TODO: Missing implementation


velox/experimental/codegen/ast/CodegenUtils.h (2 lines):
	- line 27: /// TODO: Why is this in an anon namespace
	- line 130: // TODO: change this to be controlled in a header i.e ret CodegenType<type,


velox/vector/fuzzer/VectorFuzzer.cpp (2 lines):
	- line 283: // TODO: We should bias towards edge cases (min, max, Nan, etc).
	- line 363: // TODO: We can fuzz nulls here as well.


velox/dwio/dwrf/reader/ReaderBase.cpp (2 lines):
	- line 134: // TODO: make a config
	- line 146: // TODO: read footer from spectrum


velox/dwio/dwrf/writer/WriterContext.h (2 lines):
	- line 351: // TODO: expose config for initial guess?
	- line 508: // TODO: remove once writer code is consolidated


velox/type/Filter.h (2 lines):
	- line 241: /// TODO Check if this filter is needed. This should not be passed down.
	- line 297: /// TODO Check if this filter is needed. This should not be passed down.


velox/vector/DictionaryVector-inl.h (2 lines):
	- line 117: // TODO (T58177479) - optimize to reuse the index vector allowing us to only
	- line 121: // TODO T70734527 dealing with zero length vector


velox/parse/Expressions.cpp (2 lines):
	- line 111: // TODO: If we get this to be recursive somehow, we can save on cast function
	- line 125: // TODO: Arguably all of this could be done with just Types.


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


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


velox/experimental/codegen/benchmark/SingleOutputNotDefaultNullsBenchmark.cpp (2 lines):
	- line 162: // TODO investigate why we are slower. @ 75%
	- line 223: // TODO: add test with coalesce


velox/expression/Expr.h (2 lines):
	- line 278: // TODO make the following metadata const, e.g. call computeMetadata in the
	- line 327: // and is used also if only one Expr is to be evaluated. TODO: Rename to


velox/dwio/parquet/reader/ParquetReader.cpp (2 lines):
	- line 262: // TODO Implement.
	- line 291: // TODO: implement proper stats


velox/functions/prestosql/StringFunctions.cpp (2 lines):
	- line 416: // TODO: analyze other options for enabling inplace i.e.:
	- line 464: // TODO: The propagation is a safe approximation here, it might be better


velox/expression/Expr.cpp (2 lines):
	- line 244: // TODO: Re-work the logic of deciding when to load which field.
	- line 1130: // TODO Move is available only for flat vectors. Check if tempResult is


velox/exec/PartitionedOutput.cpp (2 lines):
	- line 38: // TODO Add support for serializing partial ranges if
	- line 157: // TODO Report outputBytes as bytes after serialization


velox/dwio/common/ColumnSelector.h (2 lines):
	- line 133: * TODO (cao) - eventually these two mappings can be replaced by expression
	- line 331: // TODO (cao) - should we have project order on node level


velox/connectors/hive/FileHandle.cpp (2 lines):
	- line 25: // TODO: remember to add in the size of the hash map and its contents
	- line 48: // TODO: build the hash map/etc per file type -- presumably after reading


velox/row/UnsafeRowDynamicSerializer.h (2 lines):
	- line 253: /// // TODO: ComplexVector.h should contain asserts for these assumptions
	- line 261: // TODO: This function is untested, please add a test case to


velox/row/UnsafeRowSerializer.h (2 lines):
	- line 190: // TODO: refactor to merge the decoding of a vector of fixed length,
	- line 449: // TODO: Row type serializer is not tested


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


velox/type/Variant.h (1 line):
	- line 558: // TODO: it'd be more efficient to put union here if it ever becomes a problem


velox/dwio/dwrf/reader/SelectiveColumnReader.cpp (1 line):
	- line 50: // TODO: why is data type instead of requested type passed in?


velox/experimental/codegen/compiler_utils/ICompiledCall.h (1 line):
	- line 49: // TODO: Missing tests


velox/functions/prestosql/aggregates/ApproxPercentileAggregate.cpp (1 line):
	- line 456: // TODO Add support for accuracy parameter


velox/exec/MergeJoin.h (1 line):
	- line 166: // TODO: Copying is inefficient especially for complex type values. Consider


velox/experimental/codegen/benchmark/BooleanBenchmarks.cpp (1 line):
	- line 263: // TODO we should do better for such deep expression


velox/functions/prestosql/Arithmetic.h (1 line):
	- line 282: // TODO: These checks are costing ~13% performance penalty, it would be nice


velox/duckdb/conversion/DuckWrapper.cpp (1 line):
	- line 154: // TODO Figure out how to perform a zero-copy conversion.


velox/dwio/dwrf/common/RLEv2.h (1 line):
	- line 117: // TODO: unroll to improve performance


velox/vector/DictionaryVector.h (1 line):
	- line 120: // TODO (T61713241): Remove this later.


velox/exec/HashProbe.h (1 line):
	- line 75: // TODO: Define batch size as bytes based on RowContainer row sizes.


velox/dwio/dwrf/common/IntEncoder.h (1 line):
	- line 332: // TODO: writeVuLong should ideally be using uint64_t as input.


velox/exec/TableWriter.cpp (1 line):
	- line 110: // TODO Find a way to not have this Presto-specific logic in here.


velox/dwio/dwrf/common/EncryptionSpecification.h (1 line):
	- line 34: // TODO: support fields other than top level ones


velox/functions/prestosql/aggregates/BoolAggregates.cpp (1 line):
	- line 181: // TODO Fix signature to match Presto.


velox/expression/VectorFunction.h (1 line):
	- line 173: // TODO: Use folly::Singleton here


velox/expression/ComplexViewTypes.h (1 line):
	- line 562: // TODO: compare is not defined for view types yet


velox/type/StringView.h (1 line):
	- line 40: // TODO: Extend the interface to parity with folly::StringPiece as needed.


velox/dwio/dwrf/reader/BinaryStreamReader.cpp (1 line):
	- line 34: : preload_(true), // TODO: is preload required ?


velox/common/file/File.cpp (1 line):
	- line 136: // TODO: does FILE really not use any more memory? From the stdio.h


velox/dwio/dwrf/reader/SelectiveColumnReader.h (1 line):
	- line 270: // to rows. TODO: Consider isFinal as template parameter.


velox/exec/TableScan.h (1 line):
	- line 43: // TODO Consult with the connector. Return true only if connector can accept


velox/experimental/codegen/external_process/ExternalProcessException.h (1 line):
	- line 24: // TODO: Investigate if std::sysem_error  would be a better choice here


velox/functions/prestosql/aggregates/BitwiseAggregates.cpp (1 line):
	- line 152: // TODO Fix the signatures to match Presto.


velox/experimental/codegen/compiler_utils/LibraryDescriptor.h (1 line):
	- line 30: // TODO: Use Aggregate initialization C++20 instead of fluent interface


velox/experimental/codegen/compiler_utils/Compiler.h (1 line):
	- line 163: // TODO: Move printing err files in external_process


velox/exec/VectorHasher.cpp (1 line):
	- line 553: // TODO Add support for strings.


velox/type/Timestamp.cpp (1 line):
	- line 88: // TODO Implement


velox/functions/prestosql/json/JsonExtractor.cpp (1 line):
	- line 55: // TODO: Blindly evict the first one, use better policy


velox/exec/MergeSource.cpp (1 line):
	- line 231: // TODO Finish the pipeline early and avoid unnecessary computing.


velox/substrait/SubstraitToVeloxExpr.cpp (1 line):
	- line 33: // TODO: Get the input type and support different types here.


velox/common/caching/CachedFactory.h (1 line):
	- line 328: // TODO: consider using folly/ScopeGuard here.


velox/experimental/codegen/Codegen.cpp (1 line):
	- line 89: // TODO: we want to register UDFs via config files rather than in code


velox/expression/CastExpr.cpp (1 line):
	- line 212: // @TODO Add support for needed complex types in T74045702


velox/vector/FlatVector-inl.h (1 line):
	- line 329: // TODO Optimization: check and remove string buffers not referenced by


velox/functions/lib/DynamicFlatVector.h (1 line):
	- line 31: // TODO:


velox/exec/HashAggregation.cpp (1 line):
	- line 194: // TODO Figure out how to re-use 'result' safely.


velox/substrait/SubstraitUtils.cpp (1 line):
	- line 39: // TODO: Support for Struct is not fully added.


velox/functions/prestosql/ArrayFunctions.h (1 line):
	- line 46: // TODO: reuse strings once support landed.


velox/experimental/codegen/benchmark/SingleOutputDefaultNullsBenchmark.cpp (1 line):
	- line 29: // Note: TODO investigate, re-run and optimize  when input is not null.


velox/serializers/PrestoSerializer.cpp (1 line):
	- line 545: // TODO: make it efficient


velox/exec/Merge.cpp (1 line):
	- line 290: // TODO Delay this call until all input data has been processed.


velox/experimental/codegen/external_process/Filesystem.h (1 line):
	- line 32: /// TODO: Track files and cleanup after process closes


velox/experimental/codegen/ast/ConstantExpression.cpp (1 line):
	- line 36: // TODO: here constants strings are copied only when assigned to


velox/vector/SequenceVector-inl.h (1 line):
	- line 87: // TODO T70734527 dealing with zero length vector


velox/experimental/codegen/vector_function/ConcatExpression-inl.h (1 line):
	- line 130: /// TODO: It's important that this function and helper be inline


velox/dwio/dwrf/common/BufferedInput.cpp (1 line):
	- line 104: // TODO: if these allocations show up on a profile, add a 'canGet'


velox/functions/prestosql/registration/RegistrationFunctions.cpp (1 line):
	- line 19: // TODO: move functions that are not shared with spark to prestosql and avoid


velox/vector/BiasVector-inl.h (1 line):
	- line 108: // TODO T70734527 dealing with zero length vector


velox/experimental/codegen/library_loader/NativeLibraryLoaderException.h (1 line):
	- line 25: // TODO: Investigate if std::system_error  might


velox/parse/TypeResolver.cpp (1 line):
	- line 57: // TODO Replace with struct_pack


velox/vector/BaseVector.h (1 line):
	- line 460: // TODO: This method reuses ensureWritable(), which ensures that both:


velox/dwio/type/fbhive/HiveTypeParser.h (1 line):
	- line 34: // TODO : Find out what to do with these types


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


velox/dwio/common/ColumnSelector.cpp (1 line):
	- line 222: // TODO (cao) - SelectedTypeBuilder is uncessary to exist, can be killed


velox/buffer/Buffer.h (1 line):
	- line 115: // TODO: `resize` is probably a better name for this method


velox/duckdb/functions/DuckFunctions.cpp (1 line):
	- line 423: // FIXME: this binding does not need to be performed on every function call


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


velox/functions/prestosql/aggregates/MinMaxByAggregates.cpp (1 line):
	- line 504: // TODO In Presto, intermediate results are represented as a struct with


velox/functions/lib/SubscriptUtil.h (1 line):
	- line 294: // TODO: Optimize the case when indices are identity.


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


velox/dwio/dwrf/reader/FlatMapColumnReader.cpp (1 line):
	- line 455: // TODO Reuse


velox/common/process/StackTrace.cpp (1 line):
	- line 161: // TODO: lineNumbers has been disabled since 2009.


velox/functions/prestosql/DateTimeFunctions.h (1 line):
	- line 378: // TODO Add support for "week".


velox/dwio/common/MetricsLog.h (1 line):
	- line 130: // TODO: as we implement memory tracking, this should be per


velox/dwio/common/FilterNode.h (1 line):
	- line 151: * TODO (cao) Ideally this should merge with TypeWithId, but due to


velox/exec/HashBuild.cpp (1 line):
	- line 148: // TODO: Load only for active rows, except if right/full outer join.


velox/core/PlanNode.cpp (1 line):
	- line 351: // TODO Add connector details.


velox/expression/ControlExpr.cpp (1 line):
	- line 355: // TODO Revisit error handling


velox/experimental/codegen/ast/CodegenCtx.h (1 line):
	- line 30: /// TODO: Since we are now tracking inputs on a per expression basis,


velox/exec/RowContainer.h (1 line):
	- line 726: // Aggregates in payload. TODO: Separate out aggregate metadata


velox/experimental/codegen/ast/UnaryExpressions.cpp (1 line):
	- line 27: /// FIXME: the below optimization is only correct if child expr has no side


velox/core/SimpleFunctionMetadata.h (1 line):
	- line 275: // TODO: remove once old writers deprecated.


velox/experimental/codegen/CompiledExpressionAnalysis.h (1 line):
	- line 47: /// TODO: create a generic base visitor class for PlanNode


velox/dwio/common/InputStream.h (1 line):
	- line 204: // TODO: configure this accurately if it actually has impact.


velox/dwio/dwrf/writer/IntegerDictionaryEncoder.h (1 line):
	- line 282: // TODO: partially specialize for integers only.


velox/dwio/parquet/reader/duckdb/InputStreamFileSystem.h (1 line):
	- line 32: // TODO: Work with DuckDB to directly support a InputStream API.


velox/dwio/dwrf/common/BufferedInput.h (1 line):
	- line 68: // data. TODO: figure out how we can use the data cache for


velox/exec/TopN.cpp (1 line):
	- line 63: // TODO Decode keys first, then decode the rest only for passing positions


velox/functions/lib/ArrayBuilder.h (1 line):
	- line 44: // TODO:


velox/dwio/dwrf/writer/FlatMapColumnWriter.cpp (1 line):
	- line 97: // TODO: If we are writing shared dictionaries, the shared dictionary data


velox/type/Filter.cpp (1 line):
	- line 583: // TODO: Implement


velox/functions/lib/Re2Functions.cpp (1 line):
	- line 260: // TODO: Potentially re-use the string vector, not just the buffer.


velox/common/serialization/Serializable.h (1 line):
	- line 133: // TODO separate defintions of composite types from declarations.


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


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


velox/common/process/StackTrace.h (1 line):
	- line 28: // TODO: Deprecate in favor of folly::symbolizer.


velox/exec/HashProbe.cpp (1 line):
	- line 318: // TODO: Consider reuse of complex types.


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