[#4012] Add new LoaderConfigurationException to indicate bad configurations in Loader API

This commit is contained in:
lukaseder 2015-01-29 15:20:17 +01:00
parent cfa13aeaa7
commit 1fff3741bd
2 changed files with 74 additions and 0 deletions

View File

@ -0,0 +1,66 @@
/**
* Copyright (c) 2009-2015, Data Geekery GmbH (http://www.datageekery.com)
* All rights reserved.
*
* This work is dual-licensed
* - under the Apache Software License 2.0 (the "ASL")
* - under the jOOQ License and Maintenance Agreement (the "jOOQ License")
* =============================================================================
* You may choose which license applies to you:
*
* - If you're using this work with Open Source databases, you may choose
* either ASL or jOOQ License.
* - If you're using this work with at least one commercial database, you must
* choose jOOQ License
*
* For more information, please visit http://www.jooq.org/licenses
*
* Apache Software License 2.0:
* -----------------------------------------------------------------------------
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*
* jOOQ License and Maintenance Agreement:
* -----------------------------------------------------------------------------
* Data Geekery grants the Customer the non-exclusive, timely limited and
* non-transferable license to install and use the Software under the terms of
* the jOOQ License and Maintenance Agreement.
*
* This library is distributed with a LIMITED WARRANTY. See the jOOQ License
* and Maintenance Agreement for more details: http://www.jooq.org/licensing
*/
package org.jooq.exception;
import org.jooq.Loader;
/**
* The <code>LoaderConfigurationException</code> indicates that a {@link Loader}
* was executed with an illegal execution configuration.
*
* @author Lukas Eder
*/
public class LoaderConfigurationException extends DataAccessException {
/**
* Generated UID
*/
private static final long serialVersionUID = 1070990582126075536L;
/**
* Constructor for LoaderConfigurationException.
*
* @param message the detail message
*/
public LoaderConfigurationException(String message) {
super(message);
}
}

View File

@ -73,6 +73,7 @@ import org.jooq.SelectQuery;
import org.jooq.Table;
import org.jooq.TableRecord;
import org.jooq.exception.DataAccessException;
import org.jooq.exception.LoaderConfigurationException;
import org.jooq.tools.StringUtils;
import org.jooq.tools.csv.CSVParser;
import org.jooq.tools.csv.CSVReader;
@ -390,6 +391,8 @@ class LoaderImpl<R extends TableRecord<R>> implements
@Override
public final LoaderImpl<R> execute() throws IOException {
checkFlags();
if (content == CONTENT_CSV) {
executeCSV();
}
@ -406,6 +409,11 @@ class LoaderImpl<R extends TableRecord<R>> implements
return this;
}
private void checkFlags() {
if (batch != BATCH_NONE && onDuplicate == ON_DUPLICATE_KEY_IGNORE)
throw new LoaderConfigurationException("Cannot apply batch loading with onDuplicateKeyIgnore flag. Turn off either flag.");
}
private void executeJSON() throws IOException {
JSONReader reader = new JSONReader(data);