n3world commented on pull request #10202: URL: https://github.com/apache/arrow/pull/10202#issuecomment-838550657
> My main issue is that the callback adds a significant amount of complication to the parser implementation and it's not obvious it's desirable to constrain ourselves in the future with supporting this functionality. I am not sure where that leaves this PR. It seems like the only functionality you would like is adding missing values to rows with too few columns. In your opinion would it be reasonable to add two enums to the parser options one for handling too few columns and one for too many with the values ERROR, SKIP and FIX. Error will be the current behaviour, skip will not include the row but add the the row to tracker of problematic rows and fix will either append or truncate values and add the row to the list of problematic rows. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
