You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hawq.apache.org by hu...@apache.org on 2019/03/16 03:36:17 UTC

[hawq] 02/06: HAWQ-1682. Remove useless log for cloud authentication

This is an automated email from the ASF dual-hosted git repository.

huor pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hawq.git

commit 621b775129b7ad42c18cc75abaedfe91752b94ac
Author: wcl14 <wa...@126.com>
AuthorDate: Tue May 29 16:46:53 2018 +0800

    HAWQ-1682. Remove useless log for cloud authentication
---
 src/backend/commands/user.c | 7 ++++---
 src/backend/libpq/auth.c    | 5 ++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/backend/commands/user.c b/src/backend/commands/user.c
index d1da5a5..7252918 100644
--- a/src/backend/commands/user.c
+++ b/src/backend/commands/user.c
@@ -559,7 +559,7 @@ CreateRole(CreateRoleStmt *stmt)
 		char *errormsg;
 		int ret = check_authentication_from_cloud(stmt->role, password,
 				&createrole, USER_SYNC, "create", &errormsg);
-		elog(INFO, "in CreateRole, ret=%d", ret);
+		elog(DEBUG3, "in CreateRole, ret=%d", ret);
 		if (ret)
 		{
 			/*
@@ -1158,7 +1158,7 @@ AlterRole(AlterRoleStmt *stmt)
 			ret = check_authentication_from_cloud(stmt->role, NULL,
 					&new_record[Anum_pg_authid_rolcreaterole - 1], USER_SYNC,
 					"alter", &errormsg);
-			elog(INFO, "in AlterRole, ret=%d", ret);
+			elog(DEBUG3, "in AlterRole, ret=%d", ret);
 			if (ret)
 			{
 				elog(ERROR, "%s", errormsg);
@@ -1212,7 +1212,7 @@ AlterRole(AlterRoleStmt *stmt)
 			int ret;
 			ret = check_authentication_from_cloud(stmt->role, password, NULL,
 					USER_SYNC, "alter", &errormsg);
-			elog(INFO, "in AlterRole, ret=%d", ret);
+			elog(DEBUG3, "in AlterRole, ret=%d", ret);
 			if (ret)
 			{
 				elog(ERROR, "%s", errormsg);
@@ -1644,6 +1644,7 @@ DropRole(DropRoleStmt *stmt)
 			char *errormsg;
 			int ret = check_authentication_from_cloud(role, NULL, NULL,
 					USER_SYNC, "drop", &errormsg);
+			elog(DEBUG3, "in DropRole, ret=%d", ret);
 			if (ret)
 			{
 				elog(ERROR, "%s", errormsg);
diff --git a/src/backend/libpq/auth.c b/src/backend/libpq/auth.c
index 1e0e888..1475ca0 100644
--- a/src/backend/libpq/auth.c
+++ b/src/backend/libpq/auth.c
@@ -2721,7 +2721,8 @@ CheckCloudAuth(Port *port)
 
 	pg_cloud_auth = true;
 
-	elog(LOG, "in CheckCloudAuth, port->hba->cloudserver=%s, pg_cloud_clustername=%s", port->hba->cloudserver, pg_cloud_clustername);
+	elog(
+			DEBUG3, "in CheckCloudAuth, port->hba->cloudserver=%s, pg_cloud_clustername=%s", port->hba->cloudserver, pg_cloud_clustername);
 	if (!port->hba->cloudserver || port->hba->cloudserver[0] == '\0')
 	{
 		ereport(LOG,
@@ -2742,11 +2743,9 @@ CheckCloudAuth(Port *port)
 		return STATUS_ERROR;
 	}
 
-	elog(LOG, "in CheckCloudAuth, before init_cloud_curl");
 	init_cloud_curl();
 
 	char *errormsg;
-	elog(LOG, "in CheckCloudAuth, before check_authentication_from_cloud");
 	result = check_authentication_from_cloud(port->user_name, passwd, NULL,
 			AUTHENTICATION_CHECK, "", &errormsg);
 	if (result)