0.9

Adjusted Signal

0.9

Raw Pattern Score

100%

Time Factor

2026-10-31

Last Push

16.8K

Stars

Java

Language

2.0M

Lines of Code

11.9K

Files

774

Pattern Hits

2026-10-31

Scan Date

0.01

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 1.6/100 · LOW

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

Hallucinated Dependency & Supply Chain Risk 2.9/100 · LOW

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

Agent Over-Engineering Ratio 0.1/100 · LOW

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

PR Review Velocity Impact 4.1/100 · LOW

Which changes need focused human review? 774 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 99HIGH 2MEDIUM 22LOW 651

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 774 distinct pattern matches across 15 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.

Hallucination Indicators99 hits · 1145 pts

Severity	File	Line	Snippet	Context
CRITICAL	…ebook/presto/hive/metastore/TestHiveMetastoreUtil.java	63	private static final org.apache.hadoop.hive.metastore.api.Table TEST_TABLE = new org.apache.hadoop.hive.metastore.ap	CODE
CRITICAL	…ebook/presto/hive/metastore/TestHiveMetastoreUtil.java	83	private static final org.apache.hadoop.hive.metastore.api.Partition TEST_PARTITION = new org.apache.hadoop.hive.meta	CODE
CRITICAL	…ebook/presto/hive/metastore/TestHiveMetastoreUtil.java	102	private static final org.apache.hadoop.hive.metastore.api.Table TEST_TABLE_WITH_UNSUPPORTED_FIELDS = new org.apache.	CODE
CRITICAL	…ebook/presto/hive/metastore/TestHiveMetastoreUtil.java	117	private static final org.apache.hadoop.hive.metastore.api.Partition TEST_PARTITION_WITH_UNSUPPORTED_FIELDS = new org	CODE
CRITICAL	…astore/glue/converter/TestGlueStatisticsConverter.java	485	.longColumnStatisticsData(software.amazon.awssdk.services.glue.model.LongColumnStatisticsData.bu	CODE
CRITICAL	…astore/glue/converter/TestGlueStatisticsConverter.java	515	.longColumnStatisticsData(software.amazon.awssdk.services.glue.model.LongColumnStatisticsData.bu	CODE
CRITICAL	…astore/glue/converter/TestGlueStatisticsConverter.java	545	.longColumnStatisticsData(software.amazon.awssdk.services.glue.model.LongColumnStatisticsData.bu	CODE
CRITICAL	…astore/glue/converter/TestGlueStatisticsConverter.java	575	.longColumnStatisticsData(software.amazon.awssdk.services.glue.model.LongColumnStatisticsData.bu	CODE
CRITICAL⚡	…n/java/com/facebook/presto/hive/HiveStorageFormat.java	57	com.facebook.hive.orc.OrcSerde.class.getName(),	CODE
CRITICAL⚡	…n/java/com/facebook/presto/hive/HiveStorageFormat.java	58	com.facebook.hive.orc.OrcInputFormat.class.getName(),	CODE
CRITICAL⚡	…n/java/com/facebook/presto/hive/HiveStorageFormat.java	59	com.facebook.hive.orc.OrcOutputFormat.class.getName(),	CODE
CRITICAL	…book/presto/hive/metastore/glue/GlueHiveMetastore.java	876	newDataColumns.add(software.amazon.awssdk.services.glue.model.Column.builder()	CODE
CRITICAL	…/hive/metastore/glue/converter/GlueInputConverter.java	138	return software.amazon.awssdk.services.glue.model.Column.builder()	CODE
CRITICAL	…/presto/hive/metastore/thrift/ThriftMetastoreUtil.java	145	org.apache.hadoop.hive.metastore.api.Database result = new org.apache.hadoop.hive.metastore.api.Database();	CODE
CRITICAL	…/presto/hive/metastore/thrift/ThriftMetastoreUtil.java	158	org.apache.hadoop.hive.metastore.api.Table result = new org.apache.hadoop.hive.metastore.api.Table();	CODE
CRITICAL	…/presto/hive/metastore/thrift/ThriftMetastoreUtil.java	374	org.apache.hadoop.hive.metastore.api.Partition result = new org.apache.hadoop.hive.metastore.api.Partition();	CODE
CRITICAL	…c/src/test/java/com/facebook/presto/orc/OrcTester.java	2095	com.facebook.hive.orc.Reader reader = com.facebook.hive.orc.OrcFile.createReader(	CODE
CRITICAL⚡	…c/src/test/java/com/facebook/presto/orc/OrcTester.java	2536	com.facebook.hive.orc.OrcConf.setVar(jobConf, HIVE_ORC_COMPRESSION, compressionCodec.name());	CODE
CRITICAL⚡	…c/src/test/java/com/facebook/presto/orc/OrcTester.java	2537	com.facebook.hive.orc.OrcConf.setIntVar(jobConf, HIVE_ORC_ENTROPY_STRING_THRESHOLD, 1);	CODE
CRITICAL⚡	…c/src/test/java/com/facebook/presto/orc/OrcTester.java	2538	com.facebook.hive.orc.OrcConf.setIntVar(jobConf, HIVE_ORC_DICTIONARY_ENCODING_INTERVAL, 2);	CODE
CRITICAL⚡	…c/src/test/java/com/facebook/presto/orc/OrcTester.java	2539	com.facebook.hive.orc.OrcConf.setBoolVar(jobConf, HIVE_ORC_BUILD_STRIDE_DICTIONARY, true);	CODE
CRITICAL	…to/spark/execution/http/PrestoSparkHttpTaskClient.java	863	com.google.common.net.MediaType parsedMediaType = com.google.common.net.MediaType.parse(value);	CODE
CRITICAL	…m/facebook/presto/pinot/TestPinotBrokerPageSource.java	137	"{ \"exceptions\": [ { \"errorCode\": 200, \"message\": \"QueryExecutionError:\\njava.lang.NumberFor	CODE
CRITICAL	…to/teradata/functions/dateformat/DateFormatParser.java	131	DateFormat lexer = new com.facebook.presto.teradata.functions.DateFormat(new ANTLRInputStream(format));	CODE
CRITICAL	…acebook/presto/plugin/clickhouse/ClickHouseClient.java	624	com.facebook.presto.plugin.clickhouse.RemoteTableNameCacheKey cacheKey = new com.facebook.presto.plugin.	CODE
CRITICAL	…c/main/java/com/facebook/presto/druid/DruidConfig.java	121	configuration.set("fs.hdfs.impl", org.apache.hadoop.hdfs.DistributedFileSystem.class.getName());	CODE
CRITICAL	…c/main/java/com/facebook/presto/druid/DruidConfig.java	122	configuration.set("fs.file.impl", org.apache.hadoop.fs.LocalFileSystem.class.getName());	CODE
CRITICAL	…cebook/presto/flightshim/TestFlightShimTpchPlugin.java	190	assertEquals(Integer.parseInt(metadata.get(METADATA_KEY_ARROW_STATUS_CODE)), io.grpc.Status.Code.INTERNAL.va	CODE
CRITICAL	…com/facebook/presto/flightshim/FlightShimProducer.java	230	metadata.insert(METADATA_KEY_ARROW_STATUS_CODE, String.valueOf(io.grpc.Status.Code.INTERNAL.value()));	CODE
CRITICAL	…java/com/facebook/presto/iceberg/TestIcebergTypes.java	112	com.facebook.presto.testing.assertions.Assert.assertEquals(lateRows, lateRowsFromEquals);	CODE
CRITICAL	…java/com/facebook/presto/iceberg/TestIcebergTypes.java	118	com.facebook.presto.testing.assertions.Assert.assertEquals(earlyRows, earlyRowsFromEquals);	CODE
CRITICAL⚡	…ceberg/AbstractTestIcebergAnalyzeUppercaseColumns.java	207	org.apache.iceberg.types.Types.NestedField.required(1, "ID", org.apache.iceberg.types.Types.IntegerType.	CODE
CRITICAL⚡	…ceberg/AbstractTestIcebergAnalyzeUppercaseColumns.java	208	org.apache.iceberg.types.Types.NestedField.optional(2, "VALUE", org.apache.iceberg.types.Types.StringTyp	CODE
CRITICAL⚡	…ceberg/AbstractTestIcebergAnalyzeUppercaseColumns.java	214	org.apache.iceberg.types.Types.NestedField.required(1, "ID", org.apache.iceberg.types.Types.IntegerType.	CODE
CRITICAL⚡	…ceberg/AbstractTestIcebergAnalyzeUppercaseColumns.java	215	org.apache.iceberg.types.Types.NestedField.optional(2, "VALUE", org.apache.iceberg.types.Types.StringTyp	CODE
CRITICAL⚡	…ceberg/AbstractTestIcebergAnalyzeUppercaseColumns.java	216	org.apache.iceberg.types.Types.NestedField.required(3, "REGION", org.apache.iceberg.types.Types.StringTy	CODE
CRITICAL	…ceberg/AbstractTestIcebergAnalyzeUppercaseColumns.java	242	org.apache.iceberg.catalog.TableIdentifier.of(TEST_SCHEMA, tableName),	CODE
CRITICAL	…ceberg/AbstractTestIcebergAnalyzeUppercaseColumns.java	267	org.apache.iceberg.catalog.TableIdentifier.of(TEST_SCHEMA, tableName),	CODE
CRITICAL	…ebook/presto/iceberg/TestIcebergParquetSchemaUtil.java	118	assertEquals(convert(schema, "table"), org.apache.iceberg.parquet.ParquetSchemaUtil.convert(schema, "table"));	CODE
CRITICAL	…cebook/presto/iceberg/IcebergNativeCatalogFactory.java	157	configuration.set("fs.hdfs.impl", org.apache.hadoop.hdfs.DistributedFileSystem.class.getName());	CODE
CRITICAL	…cebook/presto/iceberg/IcebergNativeCatalogFactory.java	158	configuration.set("fs.file.impl", org.apache.hadoop.fs.LocalFileSystem.class.getName());	CODE
CRITICAL	…t/java/com/facebook/presto/mongodb/TestMongoViews.java	85	QueryRunner queryRunner = com.facebook.presto.tests.DistributedQueryRunner.builder(session)	CODE
CRITICAL⚡	…book/presto/plugin/prometheus/TestPrometheusSplit.java	144	com.facebook.airlift.units.Duration.timeUnitToString(config.getQueryChunkSizeDuration().getUnit());	CODE
CRITICAL⚡	…book/presto/plugin/prometheus/TestPrometheusSplit.java	151	config.setMaxQueryRangeDuration(com.facebook.airlift.units.Duration.valueOf("21d"));	CODE
CRITICAL⚡	…book/presto/plugin/prometheus/TestPrometheusSplit.java	152	config.setQueryChunkSizeDuration(com.facebook.airlift.units.Duration.valueOf("1d"));	CODE
CRITICAL⚡	…book/presto/plugin/prometheus/TestPrometheusSplit.java	153	config.setCacheDuration(com.facebook.airlift.units.Duration.valueOf("30s"));	CODE
CRITICAL	…/com/facebook/presto/hive/TestHudiDirectoryLister.java	68	hadoopConf.set("fs.hdfs.impl", org.apache.hadoop.hdfs.DistributedFileSystem.class.getName());	CODE
CRITICAL	…/com/facebook/presto/hive/TestHudiDirectoryLister.java	69	hadoopConf.set("fs.file.impl", org.apache.hadoop.fs.LocalFileSystem.class.getName());	CODE
CRITICAL	…/com/facebook/presto/hive/TestHudiDirectoryLister.java	85	hadoopConf.set("fs.hdfs.impl", org.apache.hadoop.hdfs.DistributedFileSystem.class.getName());	CODE
CRITICAL	…/com/facebook/presto/hive/TestHudiDirectoryLister.java	86	hadoopConf.set("fs.file.impl", org.apache.hadoop.fs.LocalFileSystem.class.getName());	CODE
CRITICAL	…est/java/com/facebook/presto/hive/TestHiveAnalyze.java	66	assertUpdate(format("CREATE TABLE %s (a_varchar varchar, month varchar) WITH (partitioned_by = ARRAY['month'], e	CODE
CRITICAL	…est/java/com/facebook/presto/hive/TestHiveAnalyze.java	107	assertUpdate(format("CREATE TABLE %s (a_varchar varchar, month varchar) WITH (partitioned_by = ARRAY['month'], e	CODE
CRITICAL	…k/presto/hive/TestGenericHiveRecordCursorProvider.java	168	hadoopConf.set("fs.file.impl", org.apache.hadoop.fs.LocalFileSystem.class.getName());	CODE
CRITICAL	…k/presto/hive/TestGenericHiveRecordCursorProvider.java	185	hadoopConf.set("fs.file.impl", org.apache.hadoop.fs.LocalFileSystem.class.getName());	CODE
CRITICAL	…va/com/facebook/presto/hive/parquet/ParquetTester.java	414	org.apache.parquet.hadoop.metadata.CompressionCodecName compressionCodecName = org.apache.parquet.hadoop	CODE
CRITICAL	…resto/hive/metastore/glue/TestingMetastoreObjects.java	108	return com.facebook.presto.hive.metastore.Database.builder()	CODE
CRITICAL	…resto/hive/metastore/glue/TestingMetastoreObjects.java	119	return com.facebook.presto.hive.metastore.Table.builder()	CODE
CRITICAL	…resto/hive/metastore/glue/TestingMetastoreObjects.java	134	return com.facebook.presto.hive.metastore.Partition.builder()	CODE
CRITICAL	…resto/hive/metastore/glue/TestingMetastoreObjects.java	146	return new com.facebook.presto.hive.metastore.Column("test-col" + generateRandom(), HiveType.HIVE_STRING, Option	CODE
CRITICAL	…ava/com/facebook/presto/hive/OrcFileWriterFactory.java	215	else if (com.facebook.hive.orc.OrcOutputFormat.class.getName().equals(storageFormat.getOutputFormat())) {	CODE
39 more matches not shown…

Over-Commented Block471 hits · 394 pts

Severity	File	Line	Snippet	Context
LOW	.yamlfmt.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	.pre-commit-config.yaml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	.yamllint.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…book/presto/hive/metastore/glue/GlueHiveMetastore.java	181		COMMENT
LOW	…est/java/com/facebook/presto/orc/TestStreamLayout.java	141	// Node 6, Column 1, Type: INT (absent in flat maps)	COMMENT
LOW	…com/facebook/presto/orc/stream/LongOutputStreamV2.java	641	// header to record the gap width. To deal with this case, we will save	COMMENT
LOW	…/facebook/presto/orc/writer/TimestampColumnWriter.java	161	}	COMMENT
LOW	…/facebook/presto/orc/writer/TimestampColumnWriter.java	181	// 2 0b001 123456700 (1234567 << 3) \| 0b001	COMMENT
LOW	…book/presto/orc/writer/LongDictionaryColumnWriter.java	161	}	COMMENT
LOW	…om/facebook/presto/orc/metadata/OrcMetadataReader.java	441	// the string stats are truncated at the first replacement character.	COMMENT
LOW	…presto/pinot/query/PinotFilterExpressionConverter.java	141	// Handle the binary comparison logic of <DATE/TIMESTAMP field> <binary op> <DATE/TIMESTAMP literal>.	COMMENT
LOW	…m/facebook/presto/pinot/query/PinotQueryGenerator.java	401	}	COMMENT
LOW	presto-native-execution/CMakeLists.txt	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	presto-native-execution/docker-compose.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	presto-native-execution/docker-compose.yml	41	# docker compose build centos-native-dependency	COMMENT
LOW	presto-native-execution/entrypoint.sh	1	#!/bin/sh	COMMENT
LOW	presto-native-execution/presto_cpp/CMakeLists.txt	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	presto-native-execution/presto_cpp/docs/conf.py	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	presto-native-execution/presto_cpp/docs/conf.py	21	# add these directories to sys.path here. If the directory is relative to the	COMMENT
LOW	presto-native-execution/presto_cpp/docs/conf.py	41		COMMENT
LOW	…ve-execution/presto_cpp/presto_protocol/topological.py	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…ve-execution/presto_cpp/presto_protocol/CMakeLists.txt	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…to-native-execution/presto_cpp/presto_protocol/util.py	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…execution/presto_cpp/presto_protocol/presto_protocol.h	21	#include "presto_cpp/presto_protocol/connector/iceberg/presto_protocol_iceberg.h"	COMMENT
LOW	…ecution/presto_cpp/presto_protocol/presto_protocol.cpp	21	#include "presto_cpp/presto_protocol/connector/tpcds/presto_protocol_tpcds.cpp"	COMMENT
LOW	…ecution/presto_cpp/presto_protocol/presto_protocol.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…tion/presto_cpp/presto_protocol/java-to-struct-json.py	1	#!/usr/bin/env python3	COMMENT
LOW	…presto_cpp/presto_protocol/core/presto_protocol_core.h	21	#include <exception>	COMMENT
LOW	…esto_cpp/presto_protocol/core/presto_protocol_core.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…cution/presto_cpp/presto_protocol/tests/CMakeLists.txt	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…/arrow_federation/presto_protocol_arrow_federation.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…onnector/arrow_flight/presto_protocol_arrow_flight.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…protocol/connector/iceberg/presto_protocol_iceberg.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…sto_protocol/connector/tpcds/presto_protocol_tpcds.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…resto_protocol/connector/tpch/presto_protocol_tpch.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	…resto_protocol/connector/hive/presto_protocol_hive.yml	1	# Licensed under the Apache License, Version 2.0 (the "License");	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxhash.h	161	# define XXH32_state_s XXH_IPREF(XXH32_state_s)	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxhash.h	181	/* specific declaration modes for Windows */	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxhash.h	201	* (therefore, avoid empty or numeric values).	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxhash.h	221	# define XXH64_createState XXH_NAME2(XXH_NAMESPACE, XXH64_createState)	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxhash.h	481	#ifdef XXH_NAMESPACE	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxhash.h	521	* Note: seed==0 produces the same results as XXH3_64bits().	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxhash.h	841	/* *************************************	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxhash.h	1001	# define XXH_rotl32 __builtin_rotateleft32	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxh3.h	61	/* special: when including `xxh3.h` directly, turn on XXH_INLINE_ALL */	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxh3.h	81	# define XXH_unlikely(x) __builtin_expect(x, 0)	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxh3.h	181	# if defined(__AVX2__)	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxh3.h	201	* This is for compatibility with aligned vector loads, which are usually faster.	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxh3.h	361	# endif	COMMENT
LOW	presto-native-execution/presto_cpp/external/xxh3.h	441	/* prefetch	COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	21	#include <algorithm> // all_of, find, for_each	COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	41	// SPDX-License-Identifier: MIT	COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	61	#if defined(NLOHMANN_JSON_VERSION_MAJOR) && defined(NLOHMANN_JSON_VERSION_MINOR) && defined(NLOHMANN_JSON_VERSION_PA	COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	81	#define JSON_USE_LEGACY_DISCARDED_VALUE_COMPARISON 0	COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	161	// __ _____ _____ _____	COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	181	#include <type_traits> // is_arithmetic, is_same, is_enum, underlying_type, is_convertible	COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	201	#endif	COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	221		COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	241	// SPDX-License-Identifier: MIT	COMMENT
LOW	…e-execution/presto_cpp/external/json/nlohmann/json.hpp	341	// SPDX-FileCopyrightText: 2016 - 2021 Evan Nemerson <evan@nemerson.com>	COMMENT
411 more matches not shown…

Structural Annotation Overuse36 hits · 73 pts

Severity	File	Line	Snippet	Context
LOW	…ative-execution/presto_cpp/main/PrestoExchangeSource.h	198	// NOTE: This method is normally called within callbacks. Caller should make	COMMENT
LOW	…ative-execution/presto_cpp/main/PrestoExchangeSource.h	233	// NOTE: This method is normally called within callbacks. Caller should make	COMMENT
LOW	…ative-execution/presto_cpp/main/PrestoExchangeSource.h	246	// NOTE: This method is normally called within callbacks. Caller should make	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	17	# Step 1: Generate CA private key and self-signed certificate.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	22	# Step 2: Generate server private key.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	25	# Step 3: Generate server certificate signing request (CSR).	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	30	# Step 4: Sign server CSR with the CA certificate to generate the server certificate.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	35	# Step 5: Generate client private key.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	38	# Step 6: Generate client certificate signing request (CSR).	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	42	# Step 7: Sign client CSR with the CA certificate to generate the client certificate.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	46	# Step 8: Output summary.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	53	# Step 9: Remove unused files.	COMMENT
LOW⚡	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	90	// Step 1: Declare default constructor	COMMENT
LOW⚡	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	93	// Step 2: Declare method	COMMENT
LOW	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	105	// Step 3: Implement method body	COMMENT
LOW⚡	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	116	// Step 4: Generate class	COMMENT
LOW⚡	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	122	// Step 5: Lookup MethodHandle	COMMENT
LOW⚡	…o/hive/functions/scalar/HiveScalarFunctionInvoker.java	74	// Step 1: Create function instance	COMMENT
LOW⚡	…o/hive/functions/scalar/HiveScalarFunctionInvoker.java	77	// Step 2: Initialize function	COMMENT
LOW⚡	…o/hive/functions/scalar/HiveScalarFunctionInvoker.java	83	// Step 3: Create invoker	COMMENT
LOW	…o/hive/functions/scalar/HiveScalarFunctionInvoker.java	94	// Step 4: Create ThreadLocal GenericUDF	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1904	// Step 1: create table partitioned only by day(int_load_ts)	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1909	// Step 2: insert old rows (spec-0 files: only day partition in metadata)	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1918	// Step 3: evolve the partition spec via the Iceberg Java API	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1929	// Step 4: insert new rows (spec-1 files: day + rec_source in metadata)	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1937	// Step 5: query before any rewrite	COMMENT
LOW	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1949	// Step 6: rewrite_data_files and verify	COMMENT
LOW	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	208	// Step 1: Create new PARTIAL AggregationNode on Source	COMMENT
LOW⚡	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	222	// Step 2: Create Exchange (hash partitioned by grouping keys) to shuffle partial states	COMMENT
LOW⚡	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	231	// Step 3: Create INTERMEDIATE AggregationNode below GroupId to merge partial states after shuffle	COMMENT
LOW	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	272	// Step 4: Create new GroupIdNode with INTERMEDIATE output as source	COMMENT
LOW	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	285	// Step 5: Change the original PARTIAL aggregation above GroupId to INTERMEDIATE.	COMMENT
LOW	…c/test/java/com/facebook/presto/server/TestServer.java	648	// Step 1: Simulate router POSTing query to backup server with retry header	COMMENT
LOW⚡	…c/test/java/com/facebook/presto/server/TestServer.java	670	// Step 2: Configure retry URL to point to the backup cluster's retry endpoint	COMMENT
LOW⚡	…c/test/java/com/facebook/presto/server/TestServer.java	673	// Step 3: Router redirects client to primary server with retry parameters	COMMENT
LOW	…c/test/java/com/facebook/presto/server/TestServer.java	733	// Step 4: Client follows the redirect link from primary server to backup server	COMMENT

Verbosity Indicators33 hits · 68 pts

Severity	File	Line	Snippet	Context
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	17	# Step 1: Generate CA private key and self-signed certificate.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	22	# Step 2: Generate server private key.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	25	# Step 3: Generate server certificate signing request (CSR).	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	30	# Step 4: Sign server CSR with the CA certificate to generate the server certificate.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	35	# Step 5: Generate client private key.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	38	# Step 6: Generate client certificate signing request (CSR).	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	42	# Step 7: Sign client CSR with the CA certificate to generate the client certificate.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	46	# Step 8: Output summary.	COMMENT
LOW⚡	…n/connectors/arrow_flight/tests/data/generate_certs.sh	53	# Step 9: Remove unused files.	COMMENT
LOW⚡	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	90	// Step 1: Declare default constructor	COMMENT
LOW⚡	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	93	// Step 2: Declare method	COMMENT
LOW	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	105	// Step 3: Implement method body	COMMENT
LOW⚡	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	116	// Step 4: Generate class	COMMENT
LOW⚡	…ook/presto/hive/functions/gen/ScalarMethodHandles.java	122	// Step 5: Lookup MethodHandle	COMMENT
LOW⚡	…o/hive/functions/scalar/HiveScalarFunctionInvoker.java	74	// Step 1: Create function instance	COMMENT
LOW⚡	…o/hive/functions/scalar/HiveScalarFunctionInvoker.java	77	// Step 2: Initialize function	COMMENT
LOW⚡	…o/hive/functions/scalar/HiveScalarFunctionInvoker.java	83	// Step 3: Create invoker	COMMENT
LOW	…o/hive/functions/scalar/HiveScalarFunctionInvoker.java	94	// Step 4: Create ThreadLocal GenericUDF	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1904	// Step 1: create table partitioned only by day(int_load_ts)	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1909	// Step 2: insert old rows (spec-0 files: only day partition in metadata)	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1918	// Step 3: evolve the partition spec via the Iceberg Java API	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1929	// Step 4: insert new rows (spec-1 files: day + rec_source in metadata)	COMMENT
LOW⚡	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1937	// Step 5: query before any rewrite	COMMENT
LOW	…book/presto/iceberg/TestRewriteDataFilesProcedure.java	1949	// Step 6: rewrite_data_files and verify	COMMENT
LOW	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	208	// Step 1: Create new PARTIAL AggregationNode on Source	COMMENT
LOW⚡	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	222	// Step 2: Create Exchange (hash partitioned by grouping keys) to shuffle partial states	COMMENT
LOW⚡	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	231	// Step 3: Create INTERMEDIATE AggregationNode below GroupId to merge partial states after shuffle	COMMENT
LOW	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	272	// Step 4: Create new GroupIdNode with INTERMEDIATE output as source	COMMENT
LOW	…/planner/iterative/rule/PreAggregateBeforeGroupId.java	285	// Step 5: Change the original PARTIAL aggregation above GroupId to INTERMEDIATE.	COMMENT
LOW	…c/test/java/com/facebook/presto/server/TestServer.java	648	// Step 1: Simulate router POSTing query to backup server with retry header	COMMENT
LOW⚡	…c/test/java/com/facebook/presto/server/TestServer.java	670	// Step 2: Configure retry URL to point to the backup cluster's retry endpoint	COMMENT
LOW⚡	…c/test/java/com/facebook/presto/server/TestServer.java	673	// Step 3: Router redirects client to primary server with retry parameters	COMMENT
LOW	…c/test/java/com/facebook/presto/server/TestServer.java	733	// Step 4: Client follows the redirect link from primary server to backup server	COMMENT

AI Slop Vocabulary16 hits · 46 pts

Severity	File	Line	Snippet	Context
MEDIUM	presto-native-execution/presto_cpp/main/PrestoTask.cpp	157	// To leverage this logic we need to supply Presto's operator names.	COMMENT
MEDIUM	…a/com/facebook/presto/iceberg/IcebergSplitManager.java	118	// propagate it at split level so the parquet pushdown can leverage it.	COMMENT
MEDIUM	…va/com/facebook/presto/iceberg/IcebergSplitSource.java	198	// TODO: We should leverage residual expression and convert that to TupleDomain.	COMMENT
MEDIUM	…m/facebook/presto/iceberg/IcebergAbstractMetadata.java	2626	// TODO: utilize and improve propertyMetadata encoder for correct encoding impl.	COMMENT
MEDIUM	…ava/com/facebook/presto/tests/AbstractTestQueries.java	781	// Checksums are order-independent, so they are robust to the plan differences the rule introduces.	COMMENT
MEDIUM	…/com/facebook/presto/tests/tpch/TpchIndexMetadata.java	78	// Note: technically we could more efficiently utilize the overlapped columns, but this way is simpler for now	COMMENT
MEDIUM	…facebook/presto/sql/planner/TestEqualityInference.java	304	// All scope equalities should utilize the constant if possible	COMMENT
MEDIUM	…com/facebook/presto/operator/MarkDistinctOperator.java	183	// Essentially we would love to have clean interfaces to support both pushing and pulling memory usage	COMMENT
MEDIUM	…va/com/facebook/presto/operator/RowNumberOperator.java	254	// Essentially we would love to have clean interfaces to support both pushing and pulling memory usage	COMMENT
MEDIUM	…om/facebook/presto/operator/DistinctLimitOperator.java	272	// Essentially we would love to have clean interfaces to support both pushing and pulling memory usage	COMMENT
MEDIUM	…ggregation/builder/InMemoryHashAggregationBuilder.java	369	// Essentially we would love to have clean interfaces to support both pushing and pulling memory usage	COMMENT
MEDIUM	…va/com/facebook/presto/sql/planner/PlanOptimizers.java	971	builder.add(predicatePushDown); // Run predicate push down one more time in case we can leverage new information	CODE
MEDIUM	…va/com/facebook/presto/sql/planner/PlanOptimizers.java	1129	builder.add(predicatePushDown); // Run predicate push down one more time in case we can leverage new information	CODE
MEDIUM	…lanner/iterative/properties/LogicalPropertiesImpl.java	312	// Some optimizations (e.g. canBeHomogenized) may utilize these	COMMENT
MEDIUM	…lanner/iterative/properties/LogicalPropertiesImpl.java	432	// Some optimizations (e.g. canBeHomogenized) may utilize these	COMMENT
MEDIUM	…resto/sql/analyzer/MaterializedViewQueryOptimizer.java	472	// TODO: We should be able to leverage this information in the StatementAnalyzer as well.	COMMENT

Fake / Example Data39 hits · 40 pts

Severity	File	Line	Snippet	Context
LOW	…cution/presto_cpp/main/tests/TaskUpdateRequestTest.cpp	281	{"Name", "John Doe"}, {"Age", "30"}, {"City", "New York"}};	CODE
LOW	…cution/presto_cpp/main/tests/TaskUpdateRequestTest.cpp	283	{"Name", "Jane Doe"}, {"Age", "25"}, {"City", "Los Angeles"}};	CODE
LOW	…sto/nativeworker/AbstractTestNativeGeneralQueries.java	506	"'Jane Doe'," +	CODE
LOW	…k/presto/iceberg/TestIcebergMaterializedViewsBase.java	1384	"(1, ARRAY['tag1', 'tag2'], MAP(ARRAY['key1', 'key2'], ARRAY['value1', 'value2']), ROW('123 Main St', 'N	CODE
LOW	…com/facebook/presto/iceberg/TestOutputColumnTypes.java	232	runQueryAndWaitForEvents("INSERT INTO employee_details VALUES(1, 'John Doe', ARRAY['SQL', 'Python', 'Kubernetes'	CODE
LOW	…ook/presto/iceberg/container/IcebergMinIODataLake.java	105	RequestBody.fromString("placeholder"));	CODE
LOW	…facebook/presto/hive/TestHiveIntegrationSmokeTest.java	5309	"AS SELECT 'dummy_value' as dummy_col WITH NO DATA";	CODE
LOW	…com/facebook/presto/hive/AbstractTestCteExecution.java	281	" (1, ARRAY[ROW('John Doe', 30)], from_base64('Sm9obiBEb2U=')), " +	CODE
LOW	…com/facebook/presto/hive/AbstractTestCteExecution.java	431	" (1, ARRAY[ROW('John Doe', 30)], from_base64('Sm9obiBEb2U=')), -- 'John Doe' in base64\n" +	CODE
LOW	…est/java/com/facebook/presto/matching/TestMatcher.java	47	assertMatch(any(), "John Doe");	CODE
LOW	…est/java/com/facebook/presto/matching/TestMatcher.java	52	assertNoMatch(typeOf(Integer.class), "John Doe");	CODE
LOW	…/java/com/facebook/presto/jdbc/TestJdbcConnection.java	71	return new Object[][] {{"test.com:1234"}, {"test@test.com"}};	CODE
LOW	…esto/operator/scalar/sql/TestReplaceFirstFunction.java	41	"REPLACE_FIRST('John Doe', ' ', '')",	CODE
LOW	…esto/operator/scalar/sql/TestReplaceFirstFunction.java	43	"John Doe");	CODE
LOW	…facebook/presto/sql/planner/TestPredicatePushdown.java	535	" AND l.comment = 'lorem ipsum' ")	CODE
LOW	…facebook/presto/sql/planner/TestPredicatePushdown.java	552	filter("L_COMMENT = 'lorem ipsum'",	CODE
LOW	…facebook/presto/sql/planner/TestPredicatePushdown.java	571	" AND l.comment = 'lorem ipsum' ")	CODE
LOW	…facebook/presto/sql/planner/TestPredicatePushdown.java	588	filter("L_COMMENT = 'lorem ipsum'",	CODE
LOW	…rver/security/oauth2/TestingHydraIdentityProvider.java	296	mapper.writeValueAsString(mapper.createObjectNode().put("subject", "foo@bar.	CODE
LOW	…ecurity/oauth2/BaseOAuth2AuthenticationFilterTest.java	206	.put("sub", "foo@bar.com")	CODE
LOW	…ecurity/oauth2/BaseOAuth2AuthenticationFilterTest.java	325	assertThat(claims.getSubject()).isEqualTo("foo@bar.com");	CODE
LOW⚡	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	79	body.put("sub", "user@example.com");	CODE
LOW⚡	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	80	body.put("email", "user@example.com");	CODE
LOW⚡	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	87	assertThat(successResponse.getUserInfo().getSubject().getValue()).isEqualTo("user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	136	body.put("email", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	152	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	168	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	184	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	203	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	219	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	235	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	254	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	273	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	289	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	305	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	344	body.put("sub", "user@example.com");	CODE
LOW	…urity/oauth2/TestNimbusOAuth2ClientUserInfoParser.java	360	body.put("sub", "user@example.com");	CODE
LOW	…/java/com/facebook/presto/delta/TestColumnMapping.java	100	"555-0101", LocalDate.of(2025, 1, 15));	CODE
LOW	…/java/com/facebook/presto/delta/TestColumnMapping.java	103	"555-0105", LocalDate.of(2026, 1, 20));	CODE

Hyper-Verbose Identifiers37 hits · 37 pts

Severity	File	Line	Snippet	Context
LOW	…to/spark/execution/http/PrestoSparkHttpTaskClient.java	550	private static class BytesResponseHandler	CODE
LOW	…to/spark/execution/http/PrestoSparkHttpTaskClient.java	728	public static class PageResponseHandler	CODE
LOW	…k/execution/http/server/SimpleHttpResponseHandler.java	31	public class SimpleHttpResponseHandler<T>	CODE
LOW	…ion/http/server/smile/AdaptingJsonResponseHandler.java	30	public class AdaptingJsonResponseHandler<T>	CODE
LOW	…ive-execution/presto_cpp/main/PrestoServerOperations.h	21	class ResponseHandler;	CODE
LOW	presto-native-execution/presto_cpp/main/PrestoServer.h	56	class ResponseHandler;	CODE
LOW	…sto-native-execution/presto_cpp/main/http/HttpServer.h	55	class AbstractRequestHandler : public proxygen::RequestHandler {	CODE
LOW	…sto-native-execution/presto_cpp/main/http/HttpServer.h	137	class CallbackRequestHandler : public AbstractRequestHandler {	CODE
LOW	…sto-native-execution/presto_cpp/main/http/HttpServer.h	171	class ErrorRequestHandler : public AbstractRequestHandler {	CODE
LOW	…sto-native-execution/presto_cpp/main/http/HttpClient.h	154	class ResponseHandler;	CODE
LOW	…o-native-execution/presto_cpp/main/http/HttpClient.cpp	180	class ResponseHandler : public proxygen::HTTPTransactionHandler {	CODE
LOW	…/presto_cpp/main/http/filters/tests/HttpFilterTest.cpp	25	class DummyRequestHandler : public proxygen::RequestHandler {	CODE
LOW	…c/main/java/com/facebook/presto/druid/DruidClient.java	183	private static class StreamingJsonResponseHandler	CODE
LOW	…to/functionNamespace/rest/RestSqlFunctionExecutor.java	159	public static class SqlFunctionResultResponseHandler	CODE
LOW	…book/presto/tests/querystats/HttpQueryStatsClient.java	59	private final class GetQueryStatsResponseHandler	CODE
LOW	…cebook/presto/connector/tvf/TestingTableFunctions.java	702	public static class IdentityPassThroughFunctionDataProcessor	CODE
LOW	…cebook/presto/connector/tvf/TestingTableFunctions.java	790	public static class RepeatFunctionDataProcessor	CODE
LOW	…cebook/presto/connector/tvf/TestingTableFunctions.java	884	private static class EmptyOutputDataProcessor	CODE
LOW	…cebook/presto/connector/tvf/TestingTableFunctions.java	937	private static class EmptyOutputWithPassThroughDataProcessor	CODE
LOW	…cebook/presto/connector/tvf/TestingTableFunctions.java	1063	private static class PassThroughInputDataProcessor	CODE
LOW	…cebook/presto/connector/tvf/TestingTableFunctions.java	1157	private static class TestInputDataProcessor	CODE
LOW	…va/com/facebook/presto/operator/JoinBridgeManager.java	219	private static class TaskWideInternalJoinBridgeDataManager<T extends JoinBridge>	CODE
LOW	…va/com/facebook/presto/operator/JoinBridgeManager.java	295	private static class OneToOneInternalJoinBridgeDataManager<T extends JoinBridge>	CODE
LOW	…va/com/facebook/presto/operator/JoinBridgeManager.java	395	private static class SharedInternalJoinBridgeDataManager<T extends JoinBridge>	CODE
LOW	…va/com/facebook/presto/proxy/ProxyResponseHandler.java	34	public class ProxyResponseHandler	CODE
LOW	presto-ui/src/utils.ts	182	export function addExponentiallyWeightedToHistory(value: number, valuesArray: number[]): number[] {	CODE
LOW	presto-ui/src/static/tableau/presto-connector.html	201	function addSessionParameterToForm(name, value) {	CODE
LOW	…/presto/server/smile/TestFullSmileResponseHandler.java	42	public class TestFullSmileResponseHandler	CODE
LOW	…ebook/presto/resourcemanager/ResourceManagerProxy.java	177	private static class ResponseHandler	CODE
LOW	…in/java/com/facebook/presto/server/WorkerResource.java	89	private static class StreamingJsonResponseHandler	CODE
LOW	…/facebook/presto/server/SimpleHttpResponseHandler.java	30	public class SimpleHttpResponseHandler<T>	CODE
LOW	…to/server/security/oauth2/NimbusAirliftHttpClient.java	105	public static class NimbusResponseHandler<T>	CODE
LOW	…ok/presto/server/thrift/ThriftHttpResponseHandler.java	31	public class ThriftHttpResponseHandler<T>	CODE
LOW	…sto/server/remotetask/HttpRemoteTaskWithEventLoop.java	1295	private class UpdateResponseHandler	CODE
LOW	…k/presto/server/smile/AdaptingJsonResponseHandler.java	31	public class AdaptingJsonResponseHandler<T>	CODE
LOW	…book/presto/server/smile/FullSmileResponseHandler.java	35	public class FullSmileResponseHandler<T>	CODE
LOW	…com/facebook/presto/operator/HttpRpcShuffleClient.java	125	public static class PageResponseHandler	CODE

TODO Padding18 hits · 27 pts

Severity	File	Line	Snippet	Context
LOW	…/facebook/presto/orc/TestMapFlatBatchStreamReader.java	72	// TODO: Add tests for timestamp as value type	COMMENT
LOW	…ebook/presto/orc/TestMapFlatSelectiveStreamReader.java	75	// TODO: Add tests for timestamp as value type	COMMENT
LOW	…esto/teradata/functions/TestTeradataDateFunctions.java	104	// TODO: implement this feature SWARM-355	COMMENT
LOW	…esto/teradata/functions/TestTeradataDateFunctions.java	114	// TODO: implement this feature SWARM-354	COMMENT
LOW	…esto/teradata/functions/TestTeradataDateFunctions.java	128	// TODO: implement this feature SWARM-353	COMMENT
LOW	…ook/presto/hive/functions/gen/AccumulatorCompiler.java	628	// TODO: implement masking based on maskChannel field once Window Functions support DISTINCT arguments to the fu	COMMENT
LOW	…resto/plugin/base/security/FileBasedAccessControl.java	303	// TODO: implement column level permissions	COMMENT
LOW	…rc/main/java/com/facebook/presto/spi/RecordCursor.java	46	// TODO: implement this method in subclasses and remove this default implementation	COMMENT
LOW	…a/com/facebook/presto/jdbc/PrestoDatabaseMetaData.java	1366	// TODO: implement this	COMMENT
LOW	…a/com/facebook/presto/jdbc/PrestoDatabaseMetaData.java	1374	// TODO: implement this	COMMENT
LOW	…/test/java/com/facebook/presto/block/TestMapBlock.java	82	// TODO: Add test case for a sliced MapBlock	COMMENT
LOW	…c/main/java/com/facebook/presto/type/DecimalCasts.java	479	// TODO: implement specialized version for short decimals	COMMENT
LOW	…c/main/java/com/facebook/presto/type/DecimalCasts.java	524	// TODO: implement specialized version for short decimals	COMMENT
LOW	…k/presto/operator/aggregation/AccumulatorCompiler.java	320	// TODO: implement masking based on maskChannel field once Window Functions support DISTINCT arguments to the fu	COMMENT
LOW	…m/facebook/presto/operator/scalar/StringFunctions.java	1015	// TODO: implement N arguments char concat	COMMENT
LOW	…sto/sql/planner/optimizations/PropertyDerivations.java	742	// TODO: implement full properties for local exchanges	COMMENT
LOW	…ebook/presto/sql/planner/optimizations/ShardJoins.java	187	// TODO: implement based on HBO stats	COMMENT
LOW	…rule/LeftJoinWithArrayContainsToEquiJoinCondition.java	96	// TODO: implement cost based with HBO	COMMENT

Decorative Section Separators6 hits · 18 pts

Severity	File	Line	Snippet	Context
MEDIUM	.github/workflows/prestocpp-linux-container-tests.yml	21	# -----------------------------	COMMENT
MEDIUM	.github/workflows/prestocpp-linux-container-tests.yml	23	# -----------------------------	COMMENT
MEDIUM	.github/workflows/prestocpp-linux-container-tests.yml	69	# -----------------------------	COMMENT
MEDIUM	.github/workflows/prestocpp-linux-container-tests.yml	71	# -----------------------------	COMMENT
MEDIUM	.github/workflows/prestocpp-linux-container-tests.yml	130	# -----------------------------	COMMENT
MEDIUM	.github/workflows/prestocpp-linux-container-tests.yml	132	# -----------------------------	COMMENT

Synthetic Comment Markers2 hits · 15 pts

Severity	File	Line	Snippet	Context
HIGH	…a/com/facebook/presto/hive/HivePageSourceProvider.java	857	// In nested columns, if the resolved type is same as requested type don't add the coercion mapp	COMMENT
HIGH	…c/test/java/com/facebook/presto/server/TestServer.java	291	// verify slug in nextUri is same as requested	COMMENT

Example Usage Blocks4 hits · 6 pts

Severity	File	Line	Snippet	Context
LOW	presto-native-execution/docker-compose.yml	14	# Usage:	COMMENT
LOW	presto-native-execution/docker-compose.yml	23	# Usage:	COMMENT
LOW	presto-native-execution/docker-compose.yml	40	# Usage:	COMMENT
LOW	presto-native-execution/docker-compose.yml	57	# Usage:	COMMENT

Deep Nesting5 hits · 5 pts

Severity	File	Line	Context
LOW	…tion/presto_cpp/presto_protocol/java-to-struct-json.py	92	CODE
LOW	…tion/presto_cpp/presto_protocol/java-to-struct-json.py	189	CODE
LOW	…native-execution/presto_cpp/main/thrift/thrift2json.py	61	CODE
LOW	…esto_cpp/main/thrift/presto_protocol-to-thrift-json.py	90	CODE
LOW	presto-native-execution/scripts/license-header.py	137	CODE

Redundant / Tautological Comments3 hits · 4 pts

Severity	File	Line	Snippet	Context
LOW	presto-native-execution/scripts/setup-macos.sh	22	# Set DEPENDENCY_DIR to a directory outside of Presto	COMMENT
LOW	.github/workflows/prestocpp-linux-build.yml	45	# Set compiler to GCC 14	COMMENT
LOW	…uct-tests/conf/docker/files/presto-launcher-wrapper.sh	38	# Check if Java version is 17	COMMENT

Unused Imports4 hits · 4 pts

Severity	File	Line	Context
LOW	presto-docs/src/main/sphinx/ext/download.py	16	CODE
LOW	presto-docs/src/main/sphinx/ext/download.py	18	CODE
LOW	presto-docs/src/main/sphinx/ext/pr.py	16	CODE
LOW	presto-docs/src/main/sphinx/ext/issue.py	16	CODE

Slop Phrases1 hit · 2 pts

Severity	File	Line	Snippet	Context
LOW	presto-cassandra/src/test/resources/cu-cassandra.yaml	419	# Warning: before enabling this property make sure to ntp is installed	COMMENT

prestodb/presto

Open-Source Code Quality, Security and Technical Debt Review