From fee9791b0afccd063e1cacf913519f09f4db42ff Mon Sep 17 00:00:00 2001 From: Sylvain Lebresne Date: Fri, 20 Nov 2015 15:16:10 +0100 Subject: [PATCH] Fix backward incompatibility in CqlInputFormat patch by jlewandowski; reviewed by mike_tr_adamson for CASSANDRA-10717 --- CHANGES.txt | 1 + .../cassandra/hadoop/cql3/CqlConfigHelper.java | 17 ++++++++++++++++- 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/CHANGES.txt b/CHANGES.txt index a9b3f362d0..e6f540622a 100644 --- a/CHANGES.txt +++ b/CHANGES.txt @@ -1,4 +1,5 @@ 3.0.1 + * Fix backward incompatibiliy in CqlInputFormat (CASSANDRA-10717) * Correctly preserve deletion info on updated rows when notifying indexers of single-row deletions (CASSANDRA-10694) * Notify indexers of partition delete during cleanup (CASSANDRA-10685) diff --git a/src/java/org/apache/cassandra/hadoop/cql3/CqlConfigHelper.java b/src/java/org/apache/cassandra/hadoop/cql3/CqlConfigHelper.java index fe62ea763e..5d5d3920ee 100644 --- a/src/java/org/apache/cassandra/hadoop/cql3/CqlConfigHelper.java +++ b/src/java/org/apache/cassandra/hadoop/cql3/CqlConfigHelper.java @@ -501,11 +501,26 @@ public class CqlConfigHelper return new LimitedLocalNodeFirstLocalBalancingPolicy(stickHosts); } + private static Optional getDefaultAuthProvider(Configuration conf) + { + Optional username = getStringSetting(USERNAME, conf); + Optional password = getStringSetting(PASSWORD, conf); + + if (username.isPresent() && password.isPresent()) + { + return Optional.of(new PlainTextAuthProvider(username.get(), password.get())); + } + else + { + return Optional.absent(); + } + } + private static Optional getAuthProvider(Configuration conf) { Optional authProvider = getInputNativeAuthProvider(conf); if (!authProvider.isPresent()) - return Optional.absent(); + return getDefaultAuthProvider(conf); return Optional.of(getClientAuthProvider(authProvider.get(), conf)); }