Skip to content

[Perf] fixed by index perf. added using ImmutableMap#2254

Merged
chernser merged 7 commits intomainfrom
perf_jdbc_improvements
Mar 28, 2025
Merged

[Perf] fixed by index perf. added using ImmutableMap#2254
chernser merged 7 commits intomainfrom
perf_jdbc_improvements

Conversation

@chernser
Copy link
Contributor

@chernser chernser commented Mar 28, 2025

Summary

  • Fixes performance for getting values by index in JDBC v2
  • Migrated TableSchema to use ImmutableMap from guava for index by name lookup.
  • Reimplemented TableSchema to make it immutable.

Closes

Checklist

Delete items not relevant to your PR:

  • Closes #
  • Unit and integration tests covering the common scenarios were added
  • A human-readable description of the changes was provided to include in CHANGELOG
  • For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

@chernser chernser requested a review from mzitnik March 28, 2025 04:45
@sonarqubecloud
Copy link

@Paultagoras Paultagoras self-requested a review March 28, 2025 12:38
@chernser chernser merged commit 7ce2cd8 into main Mar 28, 2025
24 of 25 checks passed
@chernser chernser deleted the perf_jdbc_improvements branch March 28, 2025 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants