Skip to content

PHOENIX-7267 CsvBulkLoadTool fails job due to a bad record with "(sta…#2399

Open
xavifeds8 wants to merge 1 commit intoapache:masterfrom
xavifeds8:PHOENIX-7267
Open

PHOENIX-7267 CsvBulkLoadTool fails job due to a bad record with "(sta…#2399
xavifeds8 wants to merge 1 commit intoapache:masterfrom
xavifeds8:PHOENIX-7267

Conversation

@xavifeds8
Copy link
Copy Markdown

…rtline 1) EOF reached before encapsulated token finished"

…rtline 1) EOF reached before encapsulated token finished"
try {
record = getLineParser().parse(value.toString());
} catch (IOException e) {
} catch (IOException | RuntimeException e) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can improve, this was a hack in commons-csv-1.0 https://github.com/apache/commons-csv/blob/4ef853bc13d3548ce9cae02c7c26ddbc1c790669/src/main/java/org/apache/commons/csv/CSVParser.java#L398

We could upgrade to newer one, where there is better error management!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants