2.4

Adjusted Signal

2.4

Raw Pattern Score

100%

Time Factor

2026-10-31

Last Push

4.2K

Stars

C++

Language

2.0M

Lines of Code

5.8K

Files

4.4K

Pattern Hits

2026-10-31

Scan Date

0.00

HC Hit Rate

Risk Review · Payload v1

Use each vector to guide manual verification of the underlying findings.

Payload schema: 1.0.0 · Engine: 2.0.0 · Scoring: heuristic-density-v1 · Scan date: 2026-10-31.

Cognitive Ownership Risk 10.6/100 · LOW

Can maintainers explain intent, assumptions, and trade-offs? 4223 finding(s).

Hallucinated Dependency & Supply Chain Risk 0.2/100 · LOW

Are imports declared and package/API names verified? 74 finding(s).

Agent Over-Engineering Ratio 0.7/100 · LOW

Are wrappers, guards, and repeated structures necessary? 95 finding(s).

PR Review Velocity Impact 11.0/100 · LOW

Which changes need focused human review? 4371 finding(s).

What These Metrics Mean

Adjusted Signal: Legacy pattern score normalized per 1,000 scanned lines and multiplied by the repository-level time factor. It is not an estimate of AI authorship.
Raw Pattern Score: Sum of severity- and context-weighted pattern findings before applying the time factor.
Time Factor: A coarse multiplier based on repository last-push metadata. It does not identify when individual code was written.
Pattern Hits: Count of configured pattern findings. Review each location and surrounding code before acting.
HC Hit Rate: High- and critical-severity findings per scanned file. Severity is a configured detector weight, not proof of a vulnerability or authorship.
Lines / Files: Scan coverage denominators used by the legacy normalized score.

Pattern-Signal History

This chart tracks the legacy adjusted pattern score across scan runs. Changes can reflect source changes, scan coverage, or detector/scoring versions; they do not measure AI adoption or prove that a team refactored code. Compare runs only when their scoring versions are compatible. The dashed line tracks the raw pattern-hit count.

Severity Breakdown

Groups findings by configured severity weight. Severity is not a confidence estimate, security rating, or proof that a package is missing. Verify imports against the project's manifests and inspect each finding in its local design context.

CRITICAL 0HIGH 27MEDIUM 69LOW 4275

Directory Score Breakdown

This horizontal bar chart shows where configured pattern scores accumulate across directories. Treat the bars as locations to inspect, not as evidence of authorship or code quality. Use the file-level findings to decide whether a directory needs a focused ownership, dependency, complexity, or review check.

Pattern Findings

The scanner identified 4371 distinct pattern matches across 19 syntactic categories. Each entry below represents a discrete location in the source code where the engine recorded a detector match to inspect in context. Expand any category row to inspect the individual file paths, line numbers, code snippets, and the lexical context (CODE, COMMENT, or STRING) in which each match was detected.

Reading the findings table: The Severity column indicates the configured finding weight (CRITICAL / HIGH / MEDIUM / LOW). The Context column identifies whether the match occurred inside executable code, an inline comment, or a string literal — legacy comment-context matches receive a ×1.5 score weight. The ⚡ bolt icon marks clustered matches: three or more patterns within a 10-line window, each receiving an additional ×1.5 density multiplier under the legacy scoring model.

Over-Commented Block3912 hits · 3876 pts

Severity	File	Line	Snippet	Context
LOW	.yamlfmt.yml	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	CMakeLists.txt	781	if(VELOX_ENABLE_REMOTE_FUNCTIONS OR VELOX_ENABLE_PARQUET OR VELOX_ENABLE_LOCAL_RUNNER_SERVICE)	COMMENT
LOW	CMakeLists.txt	961	# third dependency, FSST, is vendored under velox/external/fsst instead.	COMMENT
LOW	.pre-commit-config.yaml	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	.yamllint.yml	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	pyproject.toml	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	docker-compose.yml	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	CMake/resolve_dependency_modules/arrow/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	CMake/resolve_dependency_modules/boost/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	CMake/resolve_dependency_modules/curl/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	CMake/resolve_dependency_modules/folly/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_expression.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_debugger_runner.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_type.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/__init__.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_signatures.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_vector2.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_runner.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_vector.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_file.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_plan_builder.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_serde.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_arrow.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/test/test_types.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/pyvelox/__init__.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/pyvelox/utils/__init__.py	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	python/pyvelox/utils/run_queries.py	1	#!/usr/bin/env python3	COMMENT
LOW	python/pyvelox/utils/data_generator.py	1	#!/usr/bin/env python3	COMMENT
LOW	website/blog/authors.yml	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/ConnectorRegistry.h	21	#include <string_view>	COMMENT
LOW	velox/connectors/Connector.h	21	#include "folly/CancellationToken.h"	COMMENT
LOW	velox/connectors/Connector.h	421	/// sizes. This is better than getCompletedBytes()/getCompletedRows()	COMMENT
LOW	velox/connectors/Connector.h	461	}	COMMENT
LOW	velox/connectors/Connector.h	481	/// Request: input = [0, 1, 2, 3, 4]	COMMENT
LOW	velox/connectors/Connector.h	541	class ConnectorQueryCtx {	COMMENT
LOW	velox/connectors/Connector.h	901	return false;	COMMENT
LOW	velox/connectors/Connector.h	921	///	COMMENT
LOW	velox/connectors/fuzzer/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/fuzzer/tests/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/tests/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/tpcds/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/tpcds/tests/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/tpch/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/tpch/tests/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/hive/TableHandle.h	101	/// list of required subfields and the connector is free to prune the rest.	COMMENT
LOW	velox/connectors/hive/ExtractionUtils.h	21		COMMENT
LOW	velox/connectors/hive/FileIndexReader.cpp	121		COMMENT
LOW	velox/connectors/hive/HiveIndexSource.cpp	21	#include "velox/common/testutil/TestValue.h"	COMMENT
LOW	velox/connectors/hive/CMakeLists.txt	1	# Copyright (c) Facebook, Inc. and its affiliates.	COMMENT
LOW	velox/connectors/hive/FileDataSource.h	21	#include "velox/common/base/RandomUtil.h"	COMMENT
LOW	velox/connectors/hive/FileConnectorUtil.cpp	21		COMMENT
LOW	velox/connectors/hive/PartitionIdGenerator.h	21	namespace facebook::velox::connector::hive {	COMMENT
LOW	velox/connectors/hive/FileTableHandle.h	21	#include "velox/type/Filter.h"	COMMENT
LOW	velox/connectors/hive/HiveConfigMacrosUndef.h	21		COMMENT
LOW	velox/connectors/hive/HivePartitionName.h	41	/// Format int64_t partition values. Specialized to handle short DECIMAL type	COMMENT
LOW	velox/connectors/hive/HivePartitionName.h	61	/// Takes (value, type, columnIndex) and returns string.	COMMENT
LOW	velox/connectors/hive/HivePartitionName.h	81	/// @param partitionKeyAsLowerCase Controls whether partition column names	COMMENT
3852 more matches not shown…

Structural Annotation Overuse131 hits · 247 pts

Severity	File	Line	Snippet	Context
LOW	.claude/skills/pr-review/SKILL.md	70	### Step 1: Read Project Guidelines	COMMENT
LOW	.claude/skills/pr-review/SKILL.md	85	### Step 2: Analyze Changes and Prior Review	COMMENT
LOW	.claude/skills/pr-review/SKILL.md	102	### Step 3: Check PR Title and Description Quality	COMMENT
LOW	.claude/skills/pr-review/SKILL.md	124	### Step 4: Deep Review	COMMENT
LOW⚡	velox/connectors/hive/tests/HiveConnectorTest.cpp	953	// Step 1: Simulate DWIO-level IoStatistics with an inaccurate estimate.	COMMENT
LOW⚡	velox/connectors/hive/tests/HiveConnectorTest.cpp	957	// Step 2: Simulate ReadFile-layer IoStats with ground-truth values.	COMMENT
LOW⚡	velox/connectors/hive/iceberg/IcebergDataSink.cpp	554	// Step 1: Apply transforms to input partition columns.	COMMENT
LOW⚡	velox/connectors/hive/iceberg/IcebergDataSink.cpp	557	// Step 2: Create RowVector based on transformed columns.	COMMENT
LOW⚡	…ox/connectors/hive/iceberg/tests/PartitionNameTest.cpp	58	// Step 1: Build transform expressions and create evaluator.	COMMENT
LOW⚡	…ox/connectors/hive/iceberg/tests/PartitionNameTest.cpp	67	// Step 2: Apply transforms to input partition columns.	COMMENT
LOW⚡	…ox/connectors/hive/iceberg/tests/PartitionNameTest.cpp	85	// Step 3: Create RowVector based on transformed columns.	COMMENT
LOW⚡	…ox/connectors/hive/iceberg/tests/PartitionNameTest.cpp	93	// Step 4: Generate partition IDs from transformed data.	COMMENT
LOW	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1072	// Step 1: Create sink tasks and start them BEFORE initializeTask().	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1097	// Step 2: Wait for handshakes to be processed.	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1100	// Step 3: NOW initialize the task with broadcast mode.	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1109	// Step 4: Finalize destinations for broadcast.	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1112	// Step 5: Create and run the producer.	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1117	// Step 6: Wait for completion — without a fix this crashes (SIGSEGV).	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1123	// Step 7: Verify all sinks received correct row counts.	COMMENT
LOW	velox/experimental/cudf/exec/CudfHashJoin.cpp	1634	// Step 1: Inner join to get (probe_idx, build_idx) pairs where keys match.	COMMENT
LOW	velox/experimental/cudf/exec/CudfHashJoin.cpp	1658	// Step 2: Apply filter to the join pairs. INNER_JOIN mode keeps only	COMMENT
LOW⚡	velox/experimental/cudf/exec/CudfHashJoin.cpp	1683	// Step 3: Create match flags using cudf::contains. For each probe row index	COMMENT
LOW⚡	velox/experimental/cudf/exec/CudfHashJoin.cpp	1691	// Step 4: Accumulate matches across build table batches using OR.	COMMENT
LOW	velox/experimental/cudf/exec/CudfHashJoin.cpp	1704	// Step 5: Handle null-aware semantics (IN vs EXISTS).	COMMENT
LOW	velox/experimental/cudf/exec/CudfHashJoin.cpp	1952	// Step 6: Build output table with all probe columns + match column	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	278	// Step 0: elementwise ops fused with masked_select_head.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	286	// Step 1: add_sizes.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	289	// Step 2: masked_select_final.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	327	// Step 0: standalone remainder.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	331	// Step 1: add + lt fused with masked_select_head (remainder not fused).	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	340	// Step 2: add_sizes.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	343	// Step 3: masked_select_final.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	351	// Step 0: standalone remainder.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	355	// Step 1: fused masked_select with add + lt.	COMMENT
LOW	velox/experimental/torchwave/docs/users_guide.md	28	#### Step 1: Write a Test Model	COMMENT
LOW	velox/experimental/torchwave/docs/users_guide.md	70	#### Step 2: Run with executor_test	COMMENT
LOW	velox/experimental/torchwave/docs/users_guide.md	194	### Step 1: Generate a Reference Frame	COMMENT
LOW	velox/experimental/torchwave/docs/users_guide.md	206	### Step 2: Run Wave with the Reference Frame	COMMENT
LOW⚡	velox/dwio/parquet/tests/reader/BloomFilterTest.cpp	215	// Step 1: Construct a Bloom filter with 1024 bytes bitset.	COMMENT
LOW⚡	velox/dwio/parquet/tests/reader/BloomFilterTest.cpp	216	// Step 2: Insert "hello", "parquet", "bloom", "filter" to Bloom filter.	COMMENT
LOW⚡	velox/dwio/parquet/tests/reader/BloomFilterTest.cpp	217	// Step 3: Call writeTo API to write to File.	COMMENT
LOW⚡	…io/parquet/writer/arrow/util/ByteStreamSplitInternal.h	136	// Step 1: _mm_unpacklo_epi8 and mm_unpackhi_epi8:	COMMENT
LOW⚡	…io/parquet/writer/arrow/util/ByteStreamSplitInternal.h	139	// Step 3: __mm_unpacklo_epi8 and _mm_unpackhi_epi8:	COMMENT
LOW⚡	…io/parquet/writer/arrow/util/ByteStreamSplitInternal.h	141	// Step 4: __mm_unpacklo_epi64 and _mm_unpackhi_epi64:	COMMENT
LOW⚡	…ox/dwio/parquet/writer/arrow/tests/BloomFilterTest.cpp	202	// Step 1: Construct a Bloom filter with 1024 bytes bitset.	COMMENT
LOW⚡	…ox/dwio/parquet/writer/arrow/tests/BloomFilterTest.cpp	203	// Step 2: Insert "hello", "parquet", "bloom", "filter" to Bloom filter.	COMMENT
LOW⚡	…ox/dwio/parquet/writer/arrow/tests/BloomFilterTest.cpp	204	// Step 3: Call writeTo API to write to File.	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	32	### Step 1: Add the `EncodingType` Enum Value	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	56	### Step 2: Implement the Encoding	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	155	### Step 3: Define Nested Stream Identifiers (if applicable)	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	173	### Step 4: Register in `EncodingFactory` (Read + Write Path)	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	199	### Step 5: Register in `EncodingUtils.h` (Selective Reader Dispatch)	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	217	### Step 6: Add Size Estimation (for Encoding Selection)	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	236	### Step 7: Enable in Encoding Selection Policy	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	266	### Step 8: Update Build Files	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	291	### Step 9: Write Tests	COMMENT
LOW	velox/dwio/nimble/encodings/ADDING_NEW_ENCODING.md	338	### Step 10: Lint and Test	COMMENT
LOW	velox/dwio/nimble/tablet/ADDING_REQUIRED_FEATURE.md	100	### Step 1: Add the `RequiredFeature` Value	COMMENT
LOW	velox/dwio/nimble/tablet/ADDING_REQUIRED_FEATURE.md	115	### Step 2: Handle the Feature in Every Read Path	COMMENT
LOW	velox/dwio/nimble/tablet/ADDING_REQUIRED_FEATURE.md	164	### Step 3: Declare Reader Support	COMMENT
71 more matches not shown…

Verbosity Indicators97 hits · 196 pts

Severity	File	Line	Snippet	Context
LOW⚡	velox/connectors/hive/tests/HiveConnectorTest.cpp	953	// Step 1: Simulate DWIO-level IoStatistics with an inaccurate estimate.	COMMENT
LOW⚡	velox/connectors/hive/tests/HiveConnectorTest.cpp	957	// Step 2: Simulate ReadFile-layer IoStats with ground-truth values.	COMMENT
LOW⚡	velox/connectors/hive/iceberg/IcebergDataSink.cpp	554	// Step 1: Apply transforms to input partition columns.	COMMENT
LOW⚡	velox/connectors/hive/iceberg/IcebergDataSink.cpp	557	// Step 2: Create RowVector based on transformed columns.	COMMENT
LOW⚡	…ox/connectors/hive/iceberg/tests/PartitionNameTest.cpp	58	// Step 1: Build transform expressions and create evaluator.	COMMENT
LOW⚡	…ox/connectors/hive/iceberg/tests/PartitionNameTest.cpp	67	// Step 2: Apply transforms to input partition columns.	COMMENT
LOW⚡	…ox/connectors/hive/iceberg/tests/PartitionNameTest.cpp	85	// Step 3: Create RowVector based on transformed columns.	COMMENT
LOW⚡	…ox/connectors/hive/iceberg/tests/PartitionNameTest.cpp	93	// Step 4: Generate partition IDs from transformed data.	COMMENT
LOW	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1072	// Step 1: Create sink tasks and start them BEFORE initializeTask().	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1097	// Step 2: Wait for handshakes to be processed.	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1100	// Step 3: NOW initialize the task with broadcast mode.	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1109	// Step 4: Finalize destinations for broadcast.	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1112	// Step 5: Create and run the producer.	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1117	// Step 6: Wait for completion — without a fix this crashes (SIGSEGV).	COMMENT
LOW⚡	…ox/experimental/ucx-exchange/tests/UcxExchangeTest.cpp	1123	// Step 7: Verify all sinks received correct row counts.	COMMENT
LOW	velox/experimental/cudf/exec/CudfHashJoin.cpp	1634	// Step 1: Inner join to get (probe_idx, build_idx) pairs where keys match.	COMMENT
LOW	velox/experimental/cudf/exec/CudfHashJoin.cpp	1658	// Step 2: Apply filter to the join pairs. INNER_JOIN mode keeps only	COMMENT
LOW⚡	velox/experimental/cudf/exec/CudfHashJoin.cpp	1683	// Step 3: Create match flags using cudf::contains. For each probe row index	COMMENT
LOW⚡	velox/experimental/cudf/exec/CudfHashJoin.cpp	1691	// Step 4: Accumulate matches across build table batches using OR.	COMMENT
LOW	velox/experimental/cudf/exec/CudfHashJoin.cpp	1704	// Step 5: Handle null-aware semantics (IN vs EXISTS).	COMMENT
LOW	velox/experimental/cudf/exec/CudfHashJoin.cpp	1952	// Step 6: Build output table with all probe columns + match column	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	278	// Step 0: elementwise ops fused with masked_select_head.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	286	// Step 1: add_sizes.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	289	// Step 2: masked_select_final.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	327	// Step 0: standalone remainder.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	331	// Step 1: add + lt fused with masked_select_head (remainder not fused).	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	340	// Step 2: add_sizes.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	343	// Step 3: masked_select_final.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	351	// Step 0: standalone remainder.	COMMENT
LOW⚡	velox/experimental/torchwave/tests/CompileTest.cpp	355	// Step 1: fused masked_select with add + lt.	COMMENT
LOW⚡	velox/dwio/parquet/tests/reader/BloomFilterTest.cpp	215	// Step 1: Construct a Bloom filter with 1024 bytes bitset.	COMMENT
LOW⚡	velox/dwio/parquet/tests/reader/BloomFilterTest.cpp	216	// Step 2: Insert "hello", "parquet", "bloom", "filter" to Bloom filter.	COMMENT
LOW⚡	velox/dwio/parquet/tests/reader/BloomFilterTest.cpp	217	// Step 3: Call writeTo API to write to File.	COMMENT
LOW⚡	…io/parquet/writer/arrow/util/ByteStreamSplitInternal.h	136	// Step 1: _mm_unpacklo_epi8 and mm_unpackhi_epi8:	COMMENT
LOW⚡	…io/parquet/writer/arrow/util/ByteStreamSplitInternal.h	139	// Step 3: __mm_unpacklo_epi8 and _mm_unpackhi_epi8:	COMMENT
LOW⚡	…io/parquet/writer/arrow/util/ByteStreamSplitInternal.h	141	// Step 4: __mm_unpacklo_epi64 and _mm_unpackhi_epi64:	COMMENT
LOW⚡	…ox/dwio/parquet/writer/arrow/tests/BloomFilterTest.cpp	202	// Step 1: Construct a Bloom filter with 1024 bytes bitset.	COMMENT
LOW⚡	…ox/dwio/parquet/writer/arrow/tests/BloomFilterTest.cpp	203	// Step 2: Insert "hello", "parquet", "bloom", "filter" to Bloom filter.	COMMENT
LOW⚡	…ox/dwio/parquet/writer/arrow/tests/BloomFilterTest.cpp	204	// Step 3: Call writeTo API to write to File.	COMMENT
LOW⚡	…lox/dwio/nimble/selective/tests/ChunkedDecoderTest.cpp	214	// Step 1: Fill the internal buffer with 9 bytes ("abcdefghi").	COMMENT
LOW⚡	…lox/dwio/nimble/selective/tests/ChunkedDecoderTest.cpp	220	// Step 2: Consume 3 bytes. inputData_ advances to bufStart+3, inputSize_=6.	COMMENT
LOW⚡	…lox/dwio/nimble/selective/tests/ChunkedDecoderTest.cpp	225	// Step 3: Request 9 bytes total (have 6, need 3 more from the stream).	COMMENT
LOW⚡	…lox/dwio/nimble/selective/tests/ChunkedDecoderTest.cpp	257	// Step 1: Fill buffer with 30 bytes.	COMMENT
LOW⚡	…lox/dwio/nimble/selective/tests/ChunkedDecoderTest.cpp	263	// Step 2: Consume most of the buffer, leaving just 2 bytes.	COMMENT
LOW⚡	…lox/dwio/nimble/selective/tests/ChunkedDecoderTest.cpp	267	// Step 3: Request more data than trailing space allows.	COMMENT
LOW⚡	…lox/dwio/nimble/selective/tests/ChunkedDecoderTest.cpp	298	// Step 1: Fill buffer with 10 bytes to get initial allocation.	COMMENT
LOW⚡	…lox/dwio/nimble/selective/tests/ChunkedDecoderTest.cpp	304	// Step 2: Consume 5 bytes.	COMMENT
LOW⚡	…lox/dwio/nimble/selective/tests/ChunkedDecoderTest.cpp	308	// Step 3: Request much more than the current buffer capacity.	COMMENT
LOW⚡	…ox/dwio/nimble/selective/tests/ReadWithVisitorTest.cpp	3162	// Step 1: prepareRead — sets up nulls, output buffers, values capacity.	COMMENT
LOW⚡	…ox/dwio/nimble/selective/tests/ReadWithVisitorTest.cpp	3165	// Step 2: Construct ColumnVisitor with explicit template parameters.	COMMENT
LOW⚡	…ox/dwio/nimble/selective/tests/ReadWithVisitorTest.cpp	3176	// Step 3: Explicit readWithVisitor call.	COMMENT
LOW⚡	…ox/dwio/nimble/selective/tests/ReadWithVisitorTest.cpp	3180	// Step 4: Inspect reader state.	COMMENT
LOW	velox/exec/HashProbe.cpp	2417	// NOTE: for right semi and right anti join types, we need to check if	COMMENT
LOW	velox/exec/tests/TableEvolutionFuzzer.cpp	1539	// Step 1: Randomly decide whether to generate remaining filters (50% chance)	COMMENT
LOW	velox/exec/tests/TableEvolutionFuzzer.cpp	1578	// Step 2: Test setup and bucketColumnIndices generation with additional	COMMENT
LOW	velox/exec/tests/TableEvolutionFuzzer.cpp	1594	// Step 3: Create and execute write tasks	COMMENT
LOW	velox/exec/tests/TableEvolutionFuzzer.cpp	1652	// Step 4: Amortize the expensive write by running many query shapes over the	COMMENT
LOW	velox/exec/tests/TableEvolutionFuzzer.cpp	1852	// Step 5: Setup scan tasks with filters and optional aggregation pushdown	COMMENT
LOW	velox/exec/tests/TableEvolutionFuzzer.cpp	2046	// Step 6: Execute scan tasks and verify results	COMMENT
LOW⚡	velox/exec/rpc/RPCState.cpp	597	// Step 1: Try to claim a ready row.	COMMENT
37 more matches not shown…

Decorative Section Separators37 hits · 116 pts

Severity	File	Line	Snippet	Context
MEDIUM	…/encodings/benchmarks/ml_id_compression/CMakeLists.txt	28	# ---------------------------------------------------------------------------	COMMENT
MEDIUM	…/encodings/benchmarks/ml_id_compression/CMakeLists.txt	34	# ---------------------------------------------------------------------------	COMMENT
MEDIUM	…/encodings/benchmarks/ml_id_compression/CMakeLists.txt	58	# ---------------------------------------------------------------------------	COMMENT
MEDIUM	…/encodings/benchmarks/ml_id_compression/CMakeLists.txt	64	# ---------------------------------------------------------------------------	COMMENT
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	194	<div style="flex: 1; text-align: right; padding-right: 4px; color: #ccc; font-size: 14px;">┌──────────────────</	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	196	<div style="flex: 1; text-align: left; padding-left: 4px; color: #ccc; font-size: 14px;">──────────────────┐</di	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	229	<div style="text-align: left; padding-left: 0; color: #c4b5fd; font-size: 13px;">←────── same encoding</div>	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	793	<div style="display: flex; justify-content: center;"><div style="flex: 1; text-align: right; padding-right: 4px;	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	808	<div style="display: flex; justify-content: center;"><div style="flex: 1; text-align: right; padding-right: 4px;	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	882	<div style="display: flex; justify-content: center;"><div style="flex: 1; text-align: right; padding-right: 4px;	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	893	<div style="display: flex; justify-content: center;"><div style="flex: 1; text-align: right; padding-right:	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	923	<div style="display: flex; justify-content: center;"><div style="flex: 1; text-align: right; padding-right: 4px;	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	989	<div style="display: flex; justify-content: center;"><div style="flex: 1; text-align: right; padding-right: 4px;	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	1017	<div style="display: flex; justify-content: center;"><div style="flex: 1; text-align: right; padding-right: 4px;	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	1289	<pre style="font-family: monospace; font-size: 11px; line-height: 1.7; color: #555; margin: 0; white-space: pre; o	CODE
MEDIUM	…x/dwio/nimble/docs/architecture/nimble_write_path.html	1440	<div style="text-align: center; font-size: 9px; color: #999; margin-top: 4px;">◄──────────────────────────── ONE	CODE
MEDIUM⚡	velox/common/base/AsyncSource.h	223	// ┌───────┐ prepare() ┌─────────┐ success ┌──────────┐	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	224	// │ kInit │ ────────────► │ kMaking │ ──────────► │kPrepared │	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	225	// └───────┘ move() └─────────┘ └──────────┘	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	229	// │ │ ┌──────────┐ move() │ │	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	231	// │ │ └──────────┘ │ │	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	233	// │ └──────────────────────┬─────────────────────┘ │	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	235	// │ cancel() ┌───────────┐ │	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	237	// │ └───────────┘ │	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	239	// └──────────────────────────┬──────────────────────────┘	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	241	// ┌───────────┐	COMMENT
MEDIUM⚡	velox/common/base/AsyncSource.h	243	// └───────────┘	COMMENT
MEDIUM	velox/exec/rpc/tests/RpcErrorClassificationTest.cpp	133	// ── Inline payload storage ──────────────────────────────────────────────────	COMMENT
MEDIUM	velox/exec/rpc/tests/RpcErrorClassificationTest.cpp	349	// ── The unset response ──────────────────────────────────────────────────────	COMMENT
MEDIUM	velox/expression/rpc/AsyncRPCFunction.h	234	// ── PER_ROW mode ──────────────────────────────────────────────	COMMENT
MEDIUM	velox/expression/rpc/AsyncRPCFunction.h	252	// ── BATCH mode ────────────────────────────────────────────────	COMMENT
MEDIUM	velox/expression/rpc/AsyncRPCFunction.h	329	// ── Output ────────────────────────────────────────────────────	COMMENT
MEDIUM	velox/expression/rpc/AsyncRPCFunction.h	339	// ── Congestion Control ───────────────────────────────────────	COMMENT
MEDIUM	.github/workflows/selective-build-plan.yml	60	# ============================================================	COMMENT
MEDIUM	.github/workflows/selective-build-plan.yml	62	# ============================================================	COMMENT
MEDIUM	.github/workflows/selective-build-plan.yml	374	# ============================================================	COMMENT
MEDIUM	.github/workflows/selective-build-plan.yml	379	# ============================================================	COMMENT

Cross-File Repetition21 hits · 105 pts

Severity	File	Snippet	Context
HIGH	velox/docs/ext/iceberg.py	description of a general spark object. :cvar allow_nesting: class is an object that allows for nested namespaces :vartyp	STRING
HIGH	velox/docs/ext/delta.py	description of a general spark object. :cvar allow_nesting: class is an object that allows for nested namespaces :vartyp	STRING
HIGH	velox/docs/ext/spark.py	description of a general spark object. :cvar allow_nesting: class is an object that allows for nested namespaces :vartyp	STRING
HIGH	velox/docs/ext/iceberg.py	may return a prefix to put before the object name in the signature.	STRING
HIGH	velox/docs/ext/delta.py	may return a prefix to put before the object name in the signature.	STRING
HIGH	velox/docs/ext/spark.py	may return a prefix to put before the object name in the signature.	STRING
HIGH	velox/docs/ext/iceberg.py	may return true if an empty argument list is to be generated even if the document contains none.	STRING
HIGH	velox/docs/ext/delta.py	may return true if an empty argument list is to be generated even if the document contains none.	STRING
HIGH	velox/docs/ext/spark.py	may return true if an empty argument list is to be generated even if the document contains none.	STRING
HIGH	velox/docs/ext/iceberg.py	transform a spark signature into rst nodes. return (fully qualified name of the thing, classname if any). if inside a cl	STRING
HIGH	velox/docs/ext/delta.py	transform a spark signature into rst nodes. return (fully qualified name of the thing, classname if any). if inside a cl	STRING
HIGH	velox/docs/ext/spark.py	transform a spark signature into rst nodes. return (fully qualified name of the thing, classname if any). if inside a cl	STRING
HIGH	velox/docs/ext/iceberg.py	handle object nesting before content for constructs that aren't nestable, the stack is bypassed, and instead only the mo	STRING
HIGH	velox/docs/ext/delta.py	handle object nesting before content for constructs that aren't nestable, the stack is bypassed, and instead only the mo	STRING
HIGH	velox/docs/ext/spark.py	handle object nesting before content for constructs that aren't nestable, the stack is bypassed, and instead only the mo	STRING
HIGH	velox/docs/ext/iceberg.py	handle object de-nesting after content if this class is a nestable object, removing the last nested class prefix ends fu	STRING
HIGH	velox/docs/ext/delta.py	handle object de-nesting after content if this class is a nestable object, removing the last nested class prefix ends fu	STRING
HIGH	velox/docs/ext/spark.py	handle object de-nesting after content if this class is a nestable object, removing the last nested class prefix ends fu	STRING
HIGH	velox/docs/ext/iceberg.py	find a spark object for "name", perhaps using the given module and/or classname. returns a list of (name, object entry)	STRING
HIGH	velox/docs/ext/delta.py	find a spark object for "name", perhaps using the given module and/or classname. returns a list of (name, object entry)	STRING
HIGH	velox/docs/ext/spark.py	find a spark object for "name", perhaps using the given module and/or classname. returns a list of (name, object entry)	STRING

AI Slop Vocabulary21 hits · 63 pts

Severity	File	Line	Snippet	Context
MEDIUM	velox/dwio/dwrf/test/ColumnWriterTest.cpp	2570	// Normalizing the null values so that we can leverage gtest	COMMENT
MEDIUM	…/encodings/benchmarks/SubIntSplitVsOpenZLBenchmark.cpp	23	// --csv robust sweep: for every (pattern, size, method) it averages	COMMENT
MEDIUM	…o/nimble/encodings/selection/EncodingSelectionPolicy.h	445	/// Parameters are relatively robust and do not need updates unless encodings	COMMENT
MEDIUM	velox/dwio/nimble/common/Vector.h	271	// purpose, let's leverage that by using its true capacity.	COMMENT
MEDIUM	…x/dwio/nimble/selective/VariableLengthColumnReader.cpp	260	// for other reasons, we can leverage it here.	COMMENT
MEDIUM	velox/common/file/File.h	291	// aren't intended for any robust use needing a filesystem.	COMMENT
MEDIUM	velox/type/HugeInt.cpp	49	// it more robust.	COMMENT
MEDIUM	velox/exec/HashTable.cpp	1337	// - On all architectures, the universally robust way is to call	COMMENT
MEDIUM	…c/fuzzer/PrestoQueryRunnerIntermediateTypeTransforms.h	63	/// can utilize the CAST operator for correct conversion semantics. To add	COMMENT
MEDIUM	velox/exec/tests/TableEvolutionFuzzer.cpp	3112	// In a more robust implementation, we would parse the expression tree	COMMENT
MEDIUM	velox/exec/tests/TableScanTest.cpp	6644	// Set the filter conditions can't leverage the column stats.	COMMENT
MEDIUM	velox/exec/rpc/CongestionController.h	38	/// signal is the per-window minimum RTT, which is robust to per-request size	COMMENT
MEDIUM	velox/functions/prestosql/ArrayIntersectExcept.cpp	301	/// right) and leverage two sets to calculate the intersection (or except):	COMMENT
MEDIUM	velox/functions/prestosql/MapKeysByTopNValues.h	36	// utilize comparator from MapTopNFunction to sort the input map.	COMMENT
MEDIUM	velox/functions/prestosql/json/SIMDJsonExtractor.h	250	// creating a new copy with padding and utilize reusePaddedStringView() to	COMMENT
MEDIUM	velox/functions/lib/ApproxMostFrequentStreamSummary.h	64	/// serialization from old version. Essentially this means the serialization	COMMENT
MEDIUM	velox/functions/lib/MakeRowFromMap.h	129	/// pointer for use within Vector Functions. This allows it to leverage the	COMMENT
MEDIUM	velox/external/jitify/jitify.hpp	678	// TODO: This is not robust to block comments, line continuations, or tabs.	COMMENT
MEDIUM	velox/expression/tests/CastExprTest.cpp	3022	// We use bigint type with custom comparison so that we can leverage an	COMMENT
MEDIUM	velox/vector/arrow/Bridge.cpp	1805	// But this is more nuanced: for types with a list of children (like	COMMENT
MEDIUM	velox/vector/tests/SimpleVectorTest.cpp	769	// to utilize simd256_extract_value<T, i>(simdBuffer).	COMMENT

Unused Imports59 hits · 59 pts

Severity	File	Line	Context
LOW	python/pyvelox/__init__.py	15	CODE
LOW	python/pyvelox/__init__.py	16	CODE
LOW	python/pyvelox/utils/__init__.py	15	CODE
LOW	velox/experimental/torchwave/aot.py	26	CODE
LOW	velox/experimental/torchwave/multi_signature.py	29	CODE
LOW	velox/experimental/torchwave/dynamo_backend.py	24	CODE
LOW	…experimental/torchwave/tests/cat_parallel_fill_test.py	15	CODE
LOW	…erimental/torchwave/tests/test_multi_signature_aoti.py	22	CODE
LOW	velox/experimental/torchwave/tests/bincount_test.py	15	CODE
LOW	…mental/torchwave/tests/cat_alloc_group_test_run_pt2.py	15	CODE
LOW	…ntal/torchwave/tests/cat_parallel_fill_test_run_pt2.py	15	CODE
LOW	…rimental/torchwave/tests/masked_select_compose_test.py	15	CODE
LOW	…/experimental/torchwave/tests/test_torchwave_dynamo.py	23	CODE
LOW	velox/experimental/torchwave/tests/cat_nd_test.py	15	CODE
LOW	velox/experimental/torchwave/tests/select_benchmark.py	15	CODE
LOW	…experimental/torchwave/tests/index_elt_default_test.py	15	CODE
LOW	…ox/experimental/torchwave/tests/cat_nd_test_run_pt2.py	15	CODE
LOW	velox/experimental/torchwave/tests/cat_dedup_test.py	15	CODE
LOW	velox/experimental/torchwave/tests/contiguous_test.py	15	CODE
LOW	…experimental/torchwave/tests/cat_dedup_test_run_pt2.py	15	CODE
LOW	velox/experimental/torchwave/tests/stack_nd_test.py	15	CODE
LOW	…ox/experimental/torchwave/tests/cat_dynamic_nd_test.py	15	CODE
LOW	…/experimental/torchwave/tests/stack_nd_test_run_pt2.py	15	CODE
LOW	…ntal/torchwave/tests/index_elt_default_test_run_pt2.py	15	CODE
LOW	…xperimental/torchwave/tests/search_ops_test_run_pt2.py	15	CODE
LOW	…imental/torchwave/tests/cat_dynamic_nd_test_run_pt2.py	15	CODE
LOW	…/experimental/torchwave/tests/cat_strided_band_test.py	15	CODE
LOW	velox/experimental/torchwave/tests/index_tensor_test.py	15	CODE
LOW	…ox/experimental/torchwave/tests/repeat_test_run_pt2.py	15	CODE
LOW	velox/experimental/torchwave/tests/clamp_add_model.py	25	CODE
LOW	velox/experimental/torchwave/tests/search_ops_test.py	15	CODE
LOW	…ental/torchwave/tests/cat_strided_band_test_run_pt2.py	15	CODE
LOW	…/torchwave/tests/masked_select_compose_test_run_pt2.py	15	CODE
LOW	…x/experimental/torchwave/tests/cat_alloc_group_test.py	15	CODE
LOW	…erimental/torchwave/tests/index_tensor_test_run_pt2.py	15	CODE
LOW	…erimental/torchwave/tests/index_select_test_run_pt2.py	15	CODE
LOW	velox/experimental/torchwave/tests/index_select_test.py	15	CODE
LOW	…/experimental/torchwave/tests/bincount_test_run_pt2.py	15	CODE
LOW	velox/experimental/torchwave/tests/repeat_test.py	15	CODE
LOW	velox/docs/ext/iceberg.py	17	CODE
LOW	velox/docs/ext/delta.py	17	CODE
LOW	velox/docs/ext/spark.py	17	CODE
LOW	velox/docs/ext/function.py	17	CODE
LOW	scripts/tests/data/foo.expected.py	13	CODE
LOW	scripts/tests/data/foo.expected.py	14	CODE
LOW	scripts/tests/data/foo.expected.py	15	CODE
LOW	scripts/tests/data/foo.expected.py	16	CODE
LOW	scripts/tests/data/hashbang.py	3	CODE
LOW	scripts/tests/data/hashbang.py	4	CODE
LOW	scripts/tests/data/hashbang.py	5	CODE
LOW	scripts/tests/data/hashbang.py	6	CODE
LOW	scripts/tests/data/foo.py	2	CODE
LOW	scripts/tests/data/foo.py	3	CODE
LOW	scripts/tests/data/foo.py	4	CODE
LOW	scripts/tests/data/foo.py	5	CODE
LOW	scripts/tests/data/hashbang.expected.py	14	CODE
LOW	scripts/tests/data/hashbang.expected.py	15	CODE
LOW	scripts/tests/data/hashbang.expected.py	16	CODE
LOW	scripts/tests/data/hashbang.expected.py	17	CODE

Hyper-Verbose Identifiers21 hits · 23 pts

Severity	File	Line	Snippet	Context
LOW	python/test/test_expression.py	103	def test_complicated_expression(self):	CODE
LOW⚡	python/test/test_debugger_runner.py	84	def test_next_with_breakpoints(self):	CODE
LOW⚡	python/test/test_debugger_runner.py	90	def test_step_with_breakpoints(self):	CODE
LOW⚡	python/test/test_debugger_runner.py	99	def test_step_all_breakpoints(self):	CODE
LOW⚡	python/test/test_debugger_runner.py	107	def test_breakpoint_with_aggregate(self):	CODE
LOW	python/test/test_debugger_runner.py	213	def test_hook_mixed_with_breakpoint(self):	CODE
LOW	python/test/test_debugger_runner.py	232	def test_hook_inspects_vector(self):	CODE
LOW	python/test/test_debugger_runner.py	282	def test_step_with_plan_id_counts(self):	CODE
LOW	python/test/test_debugger_runner.py	304	def test_step_with_plan_id_default_behavior(self):	CODE
LOW	python/test/test_signatures.py	70	def test_aggregate_signatures(self):	CODE
LOW	python/test/test_runner.py	143	def test_mark_sorted_unsorted(self):	CODE
LOW	python/test/test_runner.py	250	def test_unnest_and_streaming_aggregate(self):	CODE
LOW	python/test/test_vector.py	378	def test_roundtrip_conversion(self):	CODE
LOW	python/test/test_vector.py	408	def test_row_vector_with_nulls(self):	CODE
LOW	python/test/test_vector.py	424	def test_row_vector_comparison(self):	CODE
LOW	python/test/test_plan_builder.py	61	def test_multiple_plan_builders(self):	CODE
LOW	velox/experimental/torchwave/lower.py	60	def package_nativert_with_torchwave_delegate(	CODE
LOW	…erimental/torchwave/tests/test_multi_signature_aoti.py	36	def test_recompile_matches_signatures(self) -> None:	CODE
LOW	…/experimental/torchwave/tests/test_torchwave_dynamo.py	71	def test_recompile_matches_signatures(self) -> None:	CODE
LOW	.github/scripts/plan-selective-build.py	194	def compute_transitive_closure(	CODE
LOW	.github/scripts/plan-selective-build.py	211	def compute_selective_build_targets(	CODE

Example Usage Blocks15 hits · 22 pts

Severity	File	Line	Snippet	Context
LOW	velox/dwio/parquet/writer/arrow/util/VisitArrayInline.h	41	/// Example usage:	COMMENT
LOW	velox/dwio/nimble/serializer/Projector.h	50	/// Example usage:	COMMENT
LOW	velox/dwio/nimble/index/HashIndex.h	51	/// Example usage:	COMMENT
LOW	velox/dwio/nimble/index/ClusterIndexBase.h	68	/// Example usage:	COMMENT
LOW	velox/serializers/KeyEncoder.h	120	/// Example usage:	COMMENT
LOW	velox/type/Variant.h	322	/// Example usage:	COMMENT
LOW	velox/type/Variant.h	377	/// Example usage:	COMMENT
LOW	velox/type/Variant.h	438	/// Example usage:	COMMENT
LOW	velox/exec/Cursor.h	130	/// Example usage:	COMMENT
LOW	velox/vector/arrow/Bridge.h	84	/// Example usage:	COMMENT
LOW	velox/vector/arrow/Bridge.h	106	/// Example usage:	COMMENT
LOW	velox/vector/arrow/Bridge.h	132	/// Example usage:	COMMENT
LOW	velox/vector/arrow/Bridge.h	158	/// Example usage:	COMMENT
LOW	.github/scripts/gh-api-retry.sh	34	# Usage:	COMMENT
LOW	.github/scripts/cmake-flags.sh	45	# Usage:	COMMENT

Empty Error Handlers4 hits · 20 pts

Severity	File	Line	Snippet	Context
HIGH	velox/docs/conf.py	33	except: # noqa E722	CODE
HIGH	velox/docs/ext/iceberg.py	266	except IndexError:	CODE
HIGH	velox/docs/ext/delta.py	266	except IndexError:	CODE
HIGH	velox/docs/ext/spark.py	266	except IndexError:	CODE

Excessive Try-Catch Wrapping9 hits · 20 pts

Severity	File	Line	Snippet	Context
MEDIUM⚡	velox/docs/credits.py	40	print("Error decoding mailmap_base64:", e)	CODE
MEDIUM⚡	velox/docs/credits.py	49	print("Error listing git log:", e)	CODE
MEDIUM⚡	velox/docs/credits.py	56	print("Error loading affiliations_map.txt:", e)	CODE
MEDIUM	velox/docs/credits.py	86	print("Error removing .mailmap:", e)	CODE
LOW	scripts/ci/benchmark-runner.py	84	except Exception:	CODE
MEDIUM	scripts/checks/check-header-ownership.py	94	print(f"Error: {velox_dir} is not a directory", file=sys.stderr)	CODE
MEDIUM	scripts/review/fetch.py	35	print(f"Error: {result.stderr.strip()}", file=sys.stderr)	CODE
MEDIUM	scripts/review/post.py	44	print(f"Error: {result.stderr.strip()}", file=sys.stderr)	CODE
MEDIUM	scripts/review/post.py	78	print("Error: review body is empty", file=sys.stderr)	CODE

Deep Nesting18 hits · 18 pts

Severity	File	Line	Context
LOW	python/pyvelox/utils/run_queries.py	264	CODE
LOW	velox/experimental/breeze/test/generator_common.py	167	CODE
LOW	…lox/experimental/breeze/test/test_fixture_generator.py	456	CODE
LOW	…lox/experimental/torchwave/tests/large_element_test.py	65	CODE
LOW	velox/experimental/torchwave/tests/element_test_100.py	70	CODE
LOW	velox/docs/ext/iceberg.py	378	CODE
LOW	velox/docs/ext/iceberg.py	547	CODE
LOW	velox/docs/ext/delta.py	376	CODE
LOW	velox/docs/ext/delta.py	545	CODE
LOW	velox/docs/ext/spark.py	376	CODE
LOW	velox/docs/ext/spark.py	545	CODE
LOW	velox/docs/ext/function.py	70	CODE
LOW	velox/docs/ext/function.py	129	CODE
LOW	velox/docs/ext/function.py	134	CODE
LOW	scripts/ci/benchmark-runner.py	89	CODE
LOW	scripts/checks/license-header.py	200	CODE
LOW	.github/scripts/generate-dependency-graph.py	155	CODE
LOW	.github/scripts/generate-dependency-graph.py	226	CODE

Fake / Example Data13 hits · 16 pts

Severity	File	Line	Snippet	Context
LOW	…/hive/storage_adapters/s3fs/tests/S3FileSystemTest.cpp	175	{{"s3.aws-secret-key", "dummy-key"},	CODE
LOW	…/hive/storage_adapters/s3fs/tests/S3FileSystemTest.cpp	176	{"s3.aws-access-key", "dummy-key"},	CODE
LOW⚡	…/hive/storage_adapters/s3fs/tests/S3FileSystemTest.cpp	232	auto s3Config = siloServer_->s3Config({{"s3.aws-access-key", "dummy-key"}});	CODE
LOW⚡	…/hive/storage_adapters/s3fs/tests/S3FileSystemTest.cpp	241	auto s3Config = siloServer_->s3Config({{"s3.aws-secret-key", "dummy-key"}});	CODE
LOW⚡	…/hive/storage_adapters/s3fs/tests/S3FileSystemTest.cpp	250	auto s3Config = siloServer_->s3Config({{"s3.aws-secret-key", "dummy-key"}});	CODE
LOW	…nnectors/hive/storage_adapters/gcs/tests/GcsEmulator.h	36	"Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor"	CODE
LOW	…nnectors/hive/storage_adapters/gcs/tests/GcsEmulator.h	36	"Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor"	CODE
LOW	…lox/dwio/parquet/tests/reader/ParquetTableScanTest.cpp	1403	{"123 Main Street", "567 Maple Drive", "789 Oak Avenue"});	CODE
LOW⚡	velox/functions/prestosql/tests/JsonFunctionsTest.cpp	691	R"({"address":{"city":"Anytown","state":"CA","street":"123 Main St","zip":"12345"},"familyMembers":[{"name":"Jane	CODE
LOW⚡	velox/functions/prestosql/tests/JsonFunctionsTest.cpp	691	R"({"address":{"city":"Anytown","state":"CA","street":"123 Main St","zip":"12345"},"familyMembers":[{"name":"Jane	CODE
LOW⚡	velox/functions/prestosql/tests/JsonFunctionsTest.cpp	691	R"({"address":{"city":"Anytown","state":"CA","street":"123 Main St","zip":"12345"},"familyMembers":[{"name":"Jane	CODE
LOW⚡	velox/functions/prestosql/tests/JsonFunctionsTest.cpp	691	R"({"address":{"city":"Anytown","state":"CA","street":"123 Main St","zip":"12345"},"familyMembers":[{"name":"Jane	CODE
LOW	velox/functions/lib/string/tests/StringImplTest.cpp	1006	runTest("John Doe", " ", "", "John Doe");	CODE

Cross-Language Confusion2 hits · 10 pts

Severity	File	Line	Snippet	Context
HIGH	python/test/test_vector.py	99	self.assertEqual(len(null), 1000)	CODE
HIGH	python/test/test_vector.py	116	for i in range(len(null)):	CODE

Slop Phrases3 hits · 8 pts

Severity	File	Line	Snippet	Context
MEDIUM	docker-compose.yml	21	# The target names are defined in docker-bake.hcl, you can use the names	COMMENT
LOW	velox/dwio/nimble/common/SchemaBuilder.h	491	// single root. We use \|roots_\| to track that the callers don't forget to	COMMENT
MEDIUM	velox/functions/tests/DynamicTypeFuncDemo.cpp	54	// For demonstration purposes, we expect the input to be a row type	COMMENT

TODO Padding2 hits · 3 pts

Severity	File	Line	Snippet	Context
LOW	velox/experimental/cudf/tests/AssignUniqueIdTest.cpp	196	// TODO: Add test for barrier execution, other operators does not support	COMMENT
LOW	velox/dwio/nimble/reader/FieldReader.cpp	1025	// TODO: implement method for bitpacked bool	COMMENT

Modern Structural Boilerplate3 hits · 3 pts

Severity	File	Line	Snippet	Context
LOW	velox/docs/ext/iceberg.py	51	logger = logging.getLogger(__name__)	CODE
LOW	velox/docs/ext/delta.py	51	logger = logging.getLogger(__name__)	CODE
LOW	velox/docs/ext/spark.py	51	logger = logging.getLogger(__name__)	CODE

Self-Referential Comments1 hit · 3 pts

Severity	File	Line	Snippet	Context
MEDIUM	scripts/ci/gersemi_cmd_definitions.py	42	# Define the argument structure of our custom functions, this influences how they are formatted	COMMENT

Redundant / Tautological Comments2 hits · 3 pts

Severity	File	Line	Snippet	Context
LOW	scripts/ci/benchmark-runner.py	198	# Read file lists from both directories.	COMMENT
LOW	.github/workflows/linux-build-base.yml	198	# Set compiler to GCC 14	COMMENT

facebookincubator/velox

Open-Source Code Quality, Security and Technical Debt Review