Skip to content

Commit 0083555

Browse files
committed
rename krb method
1 parent 8a5392f commit 0083555

5 files changed

Lines changed: 6 additions & 6 deletions

File tree

core/src/main/java/com/dtstack/flink/sql/util/KrbUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ public class KrbUtils {
4040
// public static final String FALSE_STR = "false";
4141
// public static final String SUBJECT_ONLY_KEY = "javax.security.auth.useSubjectCredsOnly";
4242

43-
public static UserGroupInformation getUgi(String principal, String keytabPath, String krb5confPath) throws IOException {
43+
public static UserGroupInformation loginAndReturnUgi(String principal, String keytabPath, String krb5confPath) throws IOException {
4444
LOG.info("Kerberos login with principal: {} and keytab: {}", principal, keytabPath);
4545
System.setProperty(KRB5_CONF_KEY, krb5confPath);
4646
// TODO 尚未探索出此选项的意义,以后研究明白方可打开

core/src/test/java/com/dtstack/flink/sql/util/KrbUtilsTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,12 @@
3030
**/
3131
public class KrbUtilsTest {
3232
@Test
33-
public void testGetUgi() throws IOException {
33+
public void testLoginAndReturnUgi() throws IOException {
3434
String principal = "";
3535
String keytabPath = "";
3636
String krb5confPath = "";
3737
try {
38-
KrbUtils.getUgi(principal, keytabPath, krb5confPath);
38+
KrbUtils.loginAndReturnUgi(principal, keytabPath, krb5confPath);
3939
} catch (IllegalArgumentException e) {
4040
Assert.assertEquals(e.getMessage(), "Can't get Kerberos realm");
4141
}

impala/impala-side/impala-all-side/src/main/java/com/dtstack/flink/sql/side/impala/ImpalaAllReqRow.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public Connection getConn(String dbUrl, String userName, String password) {
7373
String keyTabFilePath = impalaSideTableInfo.getKeyTabFilePath();
7474
String krb5FilePath = impalaSideTableInfo.getKrb5FilePath();
7575
String principal = impalaSideTableInfo.getPrincipal();
76-
UserGroupInformation ugi = KrbUtils.getUgi(principal, keyTabFilePath, krb5FilePath);
76+
UserGroupInformation ugi = KrbUtils.loginAndReturnUgi(principal, keyTabFilePath, krb5FilePath);
7777
connection = ugi.doAs(new PrivilegedExceptionAction<Connection>() {
7878
@Override
7979
public Connection run() throws SQLException {

impala/impala-side/impala-async-side/src/main/java/com/dtstack/flink/sql/side/impala/ImpalaAsyncReqRow.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ public void open(Configuration parameters) throws Exception {
7272
String keyTabFilePath = impalaSideTableInfo.getKeyTabFilePath();
7373
String krb5FilePath = impalaSideTableInfo.getKrb5FilePath();
7474
String principal = impalaSideTableInfo.getPrincipal();
75-
ugi = KrbUtils.getUgi(principal, keyTabFilePath, krb5FilePath);
75+
ugi = KrbUtils.loginAndReturnUgi(principal, keyTabFilePath, krb5FilePath);
7676
openJdbc(parameters);
7777
} else {
7878
openJdbc(parameters);

impala/impala-sink/src/main/java/com/dtstack/flink/sql/sink/impala/ImpalaOutputFormat.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ public ImpalaOutputFormat(
5555
@Override
5656
public void open(int taskNumber, int numTasks) throws IOException {
5757
if (authMech == 1) {
58-
UserGroupInformation ugi = KrbUtils.getUgi(principal, keytabPath, krb5confPath);
58+
UserGroupInformation ugi = KrbUtils.loginAndReturnUgi(principal, keytabPath, krb5confPath);
5959
try {
6060
ugi.doAs(new PrivilegedExceptionAction<Void>() {
6161
@Override

0 commit comments

Comments
 (0)