Skip to content

Commit

Permalink
Nit: Fixed comments
Browse files Browse the repository at this point in the history
  • Loading branch information
lohia-shalini committed Jan 9, 2025
1 parent bdf9548 commit 29a6f45
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions contrib/babelfishpg_tds/src/backend/tds/tdsutils.c
Original file line number Diff line number Diff line change
Expand Up @@ -935,15 +935,15 @@ check_babelfish_droprole_restrictions(char *role)
* that is only accessible in babelfish_db.
* Since role related DDLs could be executed in any PG databases,
* This function check the underlying assumption on the membership chain instead
* sysadmin <-- dbo* <--- db_owner* <--- users/roles
* bbf_admin_oid <-- dbo* <--- db_owner* <--- users/roles
*
* actual dbo and db_owner name varies across different babelfish logical databases
*/
static bool
is_babelfish_role(const char *role)
{
Oid role_oid;
Oid bbf_admin_oid;
Oid bbf_admin_oid;

role_oid = get_role_oid(role, true); /* missing OK */
bbf_admin_oid = get_role_oid(BABELFISH_ROLE_ADMIN, true); /* missing OK */
Expand All @@ -952,7 +952,7 @@ is_babelfish_role(const char *role)
if (pg_strcasecmp(role, BABELFISH_ROLE_ADMIN) == 0)
return true;

/* If a role as 'bbf_role_admin' as a member, it's a Babelfish role. */
/* If a role has 'bbf_role_admin' as a member, it's a Babelfish role. */
if (is_member_of_role(bbf_admin_oid, role_oid))
return true;

Expand Down

0 comments on commit 29a6f45

Please sign in to comment.