[PHOENIX-2239] Honor strict option when catching data error during CSV load

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.6.0
• Assignee: Gabriel Reid
• Reporter: James R. Taylor
• Created: 08/Sep/15 19:11
• Updated: 21/Nov/15 02:18
• Resolved: 19/Sep/15 18:39

Description

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:

        @Override
        public void errorOnRecord(CSVRecord csvRecord, String errorMessage) {
            LOG.error("Error upserting record {}: {}", csvRecord, errorMessage);
        }