Repository: metamodel Updated Branches: refs/heads/master 7e4ef0c3e -> 406422c67
METAMODEL-106: Added constructor args validation Project: http://git-wip-us.apache.org/repos/asf/metamodel/repo Commit: http://git-wip-us.apache.org/repos/asf/metamodel/commit/ab12454d Tree: http://git-wip-us.apache.org/repos/asf/metamodel/tree/ab12454d Diff: http://git-wip-us.apache.org/repos/asf/metamodel/diff/ab12454d Branch: refs/heads/master Commit: ab12454df7930db36a1b7503865921158a21c80e Parents: 7e4ef0c Author: Kasper Sørensen <[email protected]> Authored: Sun Jan 25 17:43:40 2015 +0100 Committer: Kasper Sørensen <[email protected]> Committed: Sun Jan 25 17:43:40 2015 +0100 ---------------------------------------------------------------------- .../metamodel/elasticsearch/ElasticSearchDataContext.java | 6 ++++++ 1 file changed, 6 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/metamodel/blob/ab12454d/elasticsearch/src/main/java/org/apache/metamodel/elasticsearch/ElasticSearchDataContext.java ---------------------------------------------------------------------- diff --git a/elasticsearch/src/main/java/org/apache/metamodel/elasticsearch/ElasticSearchDataContext.java b/elasticsearch/src/main/java/org/apache/metamodel/elasticsearch/ElasticSearchDataContext.java index 780631f..60b2274 100644 --- a/elasticsearch/src/main/java/org/apache/metamodel/elasticsearch/ElasticSearchDataContext.java +++ b/elasticsearch/src/main/java/org/apache/metamodel/elasticsearch/ElasticSearchDataContext.java @@ -103,6 +103,12 @@ public class ElasticSearchDataContext extends QueryPostprocessDataContext implem * and column model of the ElasticSearch index. */ public ElasticSearchDataContext(Client client, String indexName, SimpleTableDef... tableDefs) { + if (client == null) { + throw new IllegalArgumentException("ElasticSearch Client cannot be null"); + } + if (indexName == null || indexName.trim().length() == 0) { + throw new IllegalArgumentException("Invalid ElasticSearch Index name: " + indexName); + } this.elasticSearchClient = client; this.indexName = indexName; this.tableDefs = tableDefs;
