[
https://issues.apache.org/jira/browse/CASSANDRA-10717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15008567#comment-15008567
]
Mike Adamson commented on CASSANDRA-10717:
------------------------------------------
[~jlewandowski] I'm not that fussed because in your patch, if the
{{AuthProvider}} is set it will overwrite the {{PlainTextAuthProvider}} anyway.
The change I was proposing was to make this clearer in the code, that if the
user provided a username, password and {{AuthProvider}} then the provided
{{AuthProvider}} took precedence over the {{PlainTextAuthProvider}}.
> Changes from CASSANDRA-9353 makes Hadoop integration backward incompatible
> --------------------------------------------------------------------------
>
> Key: CASSANDRA-10717
> URL: https://issues.apache.org/jira/browse/CASSANDRA-10717
> Project: Cassandra
> Issue Type: Bug
> Reporter: Jacek Lewandowski
> Assignee: Jacek Lewandowski
> Attachments: CqlConfigHelper_authentication_patch.patch
>
>
> Previously {{CqlInputFormat.getSplits(JobContext)}} method did not require
> anything but providing username and password in order to work with password
> protected Cassandra. Now, since CASSANDRA-9353 moved this part of the code
> from Thrift to Java Driver, the user needs to explicitly select
> {{AuthProvider}} to make it work.
> I propose a simple change in {{CqlConfigHelper}} which make
> {{CqlConfigHelper.getCluster}} method set credentials on {{Cluster.Builder}}
> if they are present in the configuration, prior to setting {{AuthProvider}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
