cgivre commented on a change in pull request #2401:
URL: https://github.com/apache/drill/pull/2401#discussion_r784053795
##########
File path:
logical/src/main/java/org/apache/drill/common/logical/security/CredentialsProvider.java
##########
@@ -33,6 +37,16 @@
* Returns map with authentication credentials. Key is the credential name,
for example {@code "username"}
* and map value is corresponding credential value.
*/
+ static final Logger logger =
LoggerFactory.getLogger(CredentialsProvider.class);
+
@JsonIgnore
Map<String, String> getCredentials();
+
+ @JsonIgnore
+ default void updateCredentials(String key, String value) throws
UserException {
+ throw UserException.internalError()
+ .message("Update credential function not implemented for " + Id.NAME)
+ .build(logger);
+ }
Review comment:
Fixed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
