[
https://issues.apache.org/jira/browse/PHOENIX-2239?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Gabriel Reid updated PHOENIX-2239:
----------------------------------
Attachment: PHOENIX-2239b.patch
Thanks for catching those System.out.printlns [~jamestaylor] -- that was
actually remnants of another test that had nothing to do with this. Ugh.
Updated the patch with your advice about passing (and re-throwing) the
underlying exception as well. I'll create a separate ticket to track the MR
side of things.
> Honor strict option when catching data error during CSV load
> ------------------------------------------------------------
>
> Key: PHOENIX-2239
> URL: https://issues.apache.org/jira/browse/PHOENIX-2239
> Project: Phoenix
> Issue Type: Bug
> Reporter: James Taylor
> Attachments: PHOENIX-2239.patch, PHOENIX-2239b.patch
>
>
> The Phoenix CSV Bulk Loader has a strict option (i.e. -s or --strict) which
> is intended to abort the load if a data error is found. Looks like we're not
> passing this option through the errorOnRecord method, but I think that'd be a
> good (and likely pretty easy) improvement:
> {code}
> @Override
> public void errorOnRecord(CSVRecord csvRecord, String errorMessage) {
> LOG.error("Error upserting record {}: {}", csvRecord,
> errorMessage);
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
