Skip to content

Commit f1762aa

Browse files
authored
Merge branch 'antalya-25.6.5' into s3_hive_style_reads_and_writes_25_6_5
2 parents 3d9e365 + 89a60bd commit f1762aa

21 files changed

+10
-311
lines changed

programs/server/Server.cpp

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -156,10 +156,6 @@
156156
# include <azure/core/diagnostics/logger.hpp>
157157
#endif
158158

159-
#if USE_PARQUET
160-
# include <Processors/Formats/Impl/ParquetFileMetaDataCache.h>
161-
#endif
162-
163159

164160
#include <incbin.h>
165161
/// A minimal file used when the server is run without installation
@@ -330,7 +326,6 @@ namespace ServerSetting
330326
extern const ServerSettingsUInt64 os_cpu_busy_time_threshold;
331327
extern const ServerSettingsFloat min_os_cpu_wait_time_ratio_to_drop_connection;
332328
extern const ServerSettingsFloat max_os_cpu_wait_time_ratio_to_drop_connection;
333-
extern const ServerSettingsUInt64 input_format_parquet_metadata_cache_max_size;
334329
}
335330

336331
namespace ErrorCodes
@@ -2428,10 +2423,6 @@ try
24282423

24292424
auto replicas_reconnector = ReplicasReconnector::init(global_context);
24302425

2431-
#if USE_PARQUET
2432-
ParquetFileMetaDataCache::instance()->setMaxSizeInBytes(server_settings[ServerSetting::input_format_parquet_metadata_cache_max_size]);
2433-
#endif
2434-
24352426
/// Set current database name before loading tables and databases because
24362427
/// system logs may copy global context.
24372428
std::string default_database = server_settings[ServerSetting::default_database];

src/Access/Common/AccessType.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,6 @@ enum class AccessType : uint8_t
184184
M(SYSTEM_DROP_SCHEMA_CACHE, "SYSTEM DROP SCHEMA CACHE, DROP SCHEMA CACHE", GLOBAL, SYSTEM_DROP_CACHE) \
185185
M(SYSTEM_DROP_FORMAT_SCHEMA_CACHE, "SYSTEM DROP FORMAT SCHEMA CACHE, DROP FORMAT SCHEMA CACHE", GLOBAL, SYSTEM_DROP_CACHE) \
186186
M(SYSTEM_DROP_S3_CLIENT_CACHE, "SYSTEM DROP S3 CLIENT, DROP S3 CLIENT CACHE", GLOBAL, SYSTEM_DROP_CACHE) \
187-
M(SYSTEM_DROP_PARQUET_METADATA_CACHE, "SYSTEM DROP PARQUET METADATA CACHE", GLOBAL, SYSTEM_DROP_CACHE) \
188187
M(SYSTEM_DROP_CACHE, "DROP CACHE", GROUP, SYSTEM) \
189188
M(SYSTEM_RELOAD_CONFIG, "RELOAD CONFIG", GLOBAL, SYSTEM_RELOAD) \
190189
M(SYSTEM_RELOAD_USERS, "RELOAD USERS", GLOBAL, SYSTEM_RELOAD) \

src/Common/ProfileEvents.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1041,8 +1041,7 @@ The server successfully detected this situation and will download merged part fr
10411041
M(IndexGenericExclusionSearchAlgorithm, "Number of times the generic exclusion search algorithm is used over the index marks", ValueType::Number) \
10421042
M(ParallelReplicasQueryCount, "Number of (sub)queries executed using parallel replicas during a query execution", ValueType::Number) \
10431043
M(DistributedConnectionReconnectCount, "Number of reconnects to other servers done during distributed query execution. It can happen when a stale connection has been acquired from connection pool", ValueType::Number) \
1044-
M(ParquetMetaDataCacheHits, "Number of times the read from filesystem cache hit the cache.", ValueType::Number) \
1045-
M(ParquetMetaDataCacheMisses, "Number of times the read from filesystem cache miss the cache.", ValueType::Number) \
1044+
10461045

10471046
#ifdef APPLY_FOR_EXTERNAL_EVENTS
10481047
#define APPLY_FOR_EVENTS(M) APPLY_FOR_BUILTIN_EVENTS(M) APPLY_FOR_EXTERNAL_EVENTS(M)

src/Core/FormatFactorySettings.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1348,7 +1348,8 @@ Limits the size of the blocks formed during data parsing in input formats in byt
13481348
DECLARE(Bool, input_format_parquet_allow_geoparquet_parser, true, R"(
13491349
Use geo column parser to convert Array(UInt8) into Point/Linestring/Polygon/MultiLineString/MultiPolygon types
13501350
)", 0) \
1351-
DECLARE(Bool, input_format_parquet_use_metadata_cache, true, R"(Enable parquet file metadata caching)", 0) \
1351+
1352+
13521353
// End of FORMAT_FACTORY_SETTINGS
13531354

13541355
#define OBSOLETE_FORMAT_SETTINGS(M, ALIAS) \

src/Core/ServerSettings.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1064,7 +1064,8 @@ The policy on how to perform a scheduling of CPU slots specified by `concurrent_
10641064
See [Controlling behavior on server CPU overload](/operations/settings/server-overload) for more details.
10651065
)", 0) \
10661066
DECLARE(Float, distributed_cache_keep_up_free_connections_ratio, 0.1f, "Soft limit for number of active connection distributed cache will try to keep free. After the number of free connections goes below distributed_cache_keep_up_free_connections_ratio * max_connections, connections with oldest activity will be closed until the number goes above the limit.", 0) \
1067-
DECLARE(UInt64, input_format_parquet_metadata_cache_max_size, 500000000, "Maximum size of parquet file metadata cache", 0) \
1067+
1068+
10681069
// clang-format on
10691070

10701071
/// If you add a setting which can be updated at runtime, please update 'changeable_settings' map in dumpToSystemServerSettingsColumns below

src/Core/SettingsChangesHistory.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -182,11 +182,6 @@ const VersionToSettingsChangesMap & getSettingsChangesHistory()
182182
{"parallel_hash_join_threshold", 0, 0, "New setting"},
183183
/// Release closed. Please use 25.4
184184
});
185-
addSettingsChanges(settings_changes_history, "24.12.2.20000",
186-
{
187-
// Altinity Antalya modifications atop of 24.12
188-
{"input_format_parquet_use_metadata_cache", true, true, "New setting, turned ON by default"}, // https://github.com/Altinity/ClickHouse/pull/586
189-
});
190185
addSettingsChanges(settings_changes_history, "25.2",
191186
{
192187
/// Release closed. Please use 25.3

src/Interpreters/InterpreterSystemQuery.cpp

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,6 @@
7979
#include <Formats/ProtobufSchemas.h>
8080
#endif
8181

82-
#if USE_PARQUET
83-
#include <Processors/Formats/Impl/ParquetFileMetaDataCache.h>
84-
#endif
85-
8682
#if USE_AWS_S3
8783
#include <IO/S3/Client.h>
8884
#endif
@@ -437,16 +433,6 @@ BlockIO InterpreterSystemQuery::execute()
437433
getContext()->clearQueryResultCache(query.query_result_cache_tag);
438434
break;
439435
}
440-
case Type::DROP_PARQUET_METADATA_CACHE:
441-
{
442-
#if USE_PARQUET
443-
getContext()->checkAccess(AccessType::SYSTEM_DROP_PARQUET_METADATA_CACHE);
444-
ParquetFileMetaDataCache::instance()->clear();
445-
break;
446-
#else
447-
throw Exception(ErrorCodes::SUPPORT_IS_DISABLED, "The server was compiled without the support for Parquet");
448-
#endif
449-
}
450436
case Type::DROP_COMPILED_EXPRESSION_CACHE:
451437
#if USE_EMBEDDED_COMPILER
452438
getContext()->checkAccess(AccessType::SYSTEM_DROP_COMPILED_EXPRESSION_CACHE);
@@ -1547,7 +1533,6 @@ AccessRightsElements InterpreterSystemQuery::getRequiredAccessForDDLOnCluster()
15471533
case Type::DROP_PAGE_CACHE:
15481534
case Type::DROP_SCHEMA_CACHE:
15491535
case Type::DROP_FORMAT_SCHEMA_CACHE:
1550-
case Type::DROP_PARQUET_METADATA_CACHE:
15511536
case Type::DROP_S3_CLIENT_CACHE:
15521537
{
15531538
required_access.emplace_back(AccessType::SYSTEM_DROP_CACHE);

src/Parsers/ASTSystemQuery.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,6 @@ void ASTSystemQuery::formatImpl(WriteBuffer & ostr, const FormatSettings & setti
467467
case Type::DROP_COMPILED_EXPRESSION_CACHE:
468468
case Type::DROP_S3_CLIENT_CACHE:
469469
case Type::DROP_ICEBERG_METADATA_CACHE:
470-
case Type::DROP_PARQUET_METADATA_CACHE:
471470
case Type::RESET_COVERAGE:
472471
case Type::RESTART_REPLICAS:
473472
case Type::JEMALLOC_PURGE:

src/Parsers/ASTSystemQuery.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ class ASTSystemQuery : public IAST, public ASTQueryWithOnCluster
4242
DROP_SCHEMA_CACHE,
4343
DROP_FORMAT_SCHEMA_CACHE,
4444
DROP_S3_CLIENT_CACHE,
45-
DROP_PARQUET_METADATA_CACHE,
4645
STOP_LISTEN,
4746
START_LISTEN,
4847
RESTART_REPLICAS,

src/Processors/Formats/IInputFormat.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,9 +70,6 @@ class IInputFormat : public SourceWithKeyCondition
7070

7171
void needOnlyCount() { need_only_count = true; }
7272

73-
/// Set additional info/key/id related to underlying storage of the ReadBuffer
74-
virtual void setStorageRelatedUniqueKey(const Settings & /*settings*/, const String & /*key*/) {}
75-
7673
protected:
7774
ReadBuffer & getReadBuffer() const { chassert(in); return *in; }
7875

0 commit comments

Comments
 (0)