Croway commented on PR #25845:
URL: https://github.com/apache/camel/pull/25845#issuecomment-5452550093

   Hello @mayurbm I am not entirley sure this is actually needed. I tried to 
reproduce the stack trace with the latest camel version on main, and this is 
what I got:
   ```
   For an input like the following:
   "OrderId"x,"Name"
     1,Alice
   
     Default bulk mode:
   
     CamelExecutionException
     Caused by: java.io.UncheckedIOException:
       org.apache.commons.csv.CSVException:
       Invalid character between encapsulated token and delimiter
       at line: 1, position: 10
         at CSVParser$CSVRecordIterator.getNextRecord(CSVParser.java:234)
         at CSVParser$CSVRecordIterator.hasNext(CSVParser.java:245)
         at CsvUnmarshaller$BulkCsvUnmarshaller.asList(CsvUnmarshaller.java:122)
         at 
CsvUnmarshaller$BulkCsvUnmarshaller.unmarshal(CsvUnmarshaller.java:114)
   
     Caused by: org.apache.commons.csv.CSVException:
       Invalid character between encapsulated token and delimiter
       at line: 1, position: 10
         at Lexer.parseEncapsulatedToken(Lexer.java:370)
         at Lexer.nextToken(Lexer.java:294)
         at CSVParser.nextRecord(CSVParser.java:892)
   
     Automatic headers (useMaps=true):
   
     CamelExecutionException
     Caused by: org.apache.commons.csv.CSVException:
       Invalid character between encapsulated token and delimiter
       at line: 1, position: 10
         at Lexer.parseEncapsulatedToken(Lexer.java:370)
         at Lexer.nextToken(Lexer.java:294)
         at CSVParser.nextRecord(CSVParser.java:892)
         at CSVParser.createHeaders(CSVParser.java:609)
         at CSVParser.<init>(CSVParser.java:563)
         at CSVParser$Builder.get(CSVParser.java:171)
         at 
CsvUnmarshaller$BulkCsvUnmarshaller.unmarshal(CsvUnmarshaller.java:109)
         at CsvDataFormat.unmarshal(CsvDataFormat.java:101)
   ```
   As you can see, commons CSV 1.14.1 already describes the error, I do not 
think we need more than this.
   
   Unless there is another current-Camel use case that demonstrates 
insufficient diagnostics, I think the issue and PR should be closed.


-- 
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]

Reply via email to