atharvalade opened a new pull request, #3196: URL: https://github.com/apache/iggy/pull/3196
## Which issue does this PR close? Closes #3175 ## Rationale Unrecognized Postgres types silently coerced to String, returning opaque `InvalidRecord` on failure with no column or type context. ## What changed? The default `_` branch in `extract_column_value` attempted to read any unrecognized Postgres type as `Option<String>`, failing silently with `Error::InvalidRecord` and no indication of which column or type caused it. Added explicit handlers for DATE, TIME, INTERVAL, OID, NAME, BPCHAR, and 13 array types (_BOOL, _INT2, _INT4, _INT8, _FLOAT4, _FLOAT8, _TEXT, _VARCHAR, _CHAR, _BPCHAR, _UUID, _JSON, _JSONB). The default branch now logs a warning with column name and type before attempting String fallback, and returns `InvalidRecordValue` with a diagnostic message on failure. ## Local Execution - Passed - Pre-commit hooks ran ## AI Usage 1. Opus 4.6 2. Scaffolding, Writing comments, Writing PR Description 3. Verified via cargo check, clippy, fmt, and full test suite (24/24 pass) 4. Yes, all code can be explained -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
