zhangshenghang commented on issue #10333: URL: https://github.com/apache/seatunnel/issues/10333#issuecomment-3754896938
<!-- code-pr-reviewer --> Thanks for the report. I've verified the code paths you mentioned: **Evidence:** - `JsonPathTransform.java:162` passes `null` as `fieldName` to `converter.convert(jsonNode, null)` - `JsonToRowConverters.java:259-271` (`convertToLocalDate`) and `L278-293` (`convertToLocalDateTime`) use `fieldName` as the cache key for `DateTimeFormatter` - `JsonPathTransformTest.java` has no test coverage for `dest_type="date"` or `dest_type="timestamp"` **Likely issue:** The `null` fieldName breaks the formatter caching mechanism in `JsonToRowConverters`, causing repeated format matching on every conversion (performance degradation). There's also potential NPE risk depending on the Map implementation. **To proceed:** 1. Can you share a reproduction config or error log? 2. What is the actual failure mode—exception, incorrect result, or performance concern? 3. Are you observing this with a specific date/timestamp format? -- 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]
