thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-24 08:32:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix build, fix test

Browse Source
This commit is contained in:
Andrey Zvonov 2024-11-10 11:08:17 +00:00
parent c4fc7a1bac
commit 6e58cfc5b8
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Access/LDAPAccessStorage.cpp
View File

@ -320,7 +320,7 @@ std::set<String> LDAPAccessStorage::mapExternalRolesNoLock(const LDAPClient::Sea
std::set<String> role_names;
// If this node can't access LDAP server (or has not privileges to fetch roles) and gets empty list of external roles
if (external_roles.size() == 0)
if (external_roles.empty())
return role_names;
if (external_roles.size() != role_search_params.size())

2
tests/integration/test_ldap_external_user_directory/test.py
View File

@ -181,3 +181,5 @@ def test_push_role_to_other_nodes(ldap_cluster):
instance1.query("DROP TABLE IF EXISTS local_table SYNC")
instance2.query("DROP TABLE IF EXISTS local_table SYNC")
instance2.query("DROP ROLE IF EXISTS role_read")
delete_ldap_group(ldap_cluster, group_cn="clickhouse-role_read")