Skip to content
This repository was archived by the owner on Apr 7, 2026. It is now read-only.

fix: findColumn was case-sensitive - #4181

Closed
sakthivelmanii wants to merge 1 commit into
mainfrom
fix_case_sensitivity_in_finding_column
Closed

fix: findColumn was case-sensitive#4181
sakthivelmanii wants to merge 1 commit into
mainfrom
fix_case_sensitivity_in_finding_column

Conversation

@sakthivelmanii

Copy link
Copy Markdown
Collaborator

Description:

Currently we store all the columns in a Treemap in a case-sensitive way. In Hibernate, we can query columns using case-insensitive way.

To fix this, we will be storing all the fields in a Lowercase format. To retrieve it, we will convert the requested column name to lowercase before actually looking in the map.

@sakthivelmanii
sakthivelmanii requested review from a team October 22, 2025 11:15
@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: spanner Issues related to the googleapis/java-spanner API. labels Oct 22, 2025
@sakthivelmanii sakthivelmanii changed the title fix: Fix case sensitivity in finding struct column fix: Fix case sensitivity in findColumn Oct 22, 2025
@sakthivelmanii
sakthivelmanii force-pushed the fix_case_sensitivity_in_finding_column branch from 1a0f255 to bc374c1 Compare October 22, 2025 11:18
@olavloite olavloite changed the title fix: Fix case sensitivity in findColumn fix: findColumn was case-sensitive Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spanner Issues related to the googleapis/java-spanner API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant