Prathamesh9284 commented on issue #690: URL: https://github.com/apache/wayang/issues/690#issuecomment-3915041411
Thanks @mspruc and @zkaoudi for the additional context! Here's my plan: 1. Improve the error message at JavaCSVTableSource.java line 127 to include expected vs actual column count and a hint about the required Calcite header format (name:type). 2. Add early header validation to detect missing or malformed headers before parsing begins. 3. Address the separator inconsistency — The header is parsed by Calcite's CsvEnumerator which hardcodes commas, while data rows use Wayang's configurable separator. I'll add validation to detect this mismatch early with a clear error message. Does this approach sound right? -- 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]
