Please address the comments from this code review:

## Individual Comments

### Comment 1
<location> `ch_tools/chadmin/cli/zookeeper_group.py:644` </location>
<code_context>
-        create_zero_copy_locks(
-            ctx, disk, table_info, partition_id, part_id, replica, dry_run
-        )
+    replicas_ = replicas.split(",")
+    for replica in replicas_:
+        if replica not in replicas_list:
</code_context>

<issue_to_address>
**suggestion:** Splitting replicas by comma does not trim whitespace, which may cause issues with user input.

Use `[r.strip() for r in replicas.split(",")]` to ensure whitespace is removed from each replica, preventing issues with user input containing spaces.

```suggestion
    replicas_ = [r.strip() for r in replicas.split(",")]
```
</issue_to_address>

### Comment 2
<location> `ch_tools/chadmin/internal/utils.py:96-97` </location>
<code_context>
+    else:
+        raise RuntimeError("For using remote() table function tcp port must be defined")
+
+    replicas_str = ",".join(replicas)
+    return f"{remote_clause}('{replicas_str}', {table}, '{user_name}', '{{user_password}}')"
+
</code_context>

<issue_to_address>
**suggestion (bug_risk):** Joining replicas without handling empty or invalid entries may result in malformed remote table function calls.

Validate that the replicas list is non-empty and all entries are valid hostnames before joining.

```suggestion
    import re

    if not replicas or not all(isinstance(replica, str) and replica.strip() for replica in replicas):
        raise ValueError("Replicas list must be non-empty and contain valid hostnames.")

    hostname_regex = re.compile(
        r"^(?=.{1,253}$)(?!-)[A-Za-z0-9-]{1,63}(?<!-)(\.(?!-)[A-Za-z0-9-]{1,63}(?<!-))*$"
    )
    for replica in replicas:
        if not hostname_regex.match(replica):
            raise ValueError(f"Invalid hostname in replicas list: '{replica}'")

    replicas_str = ",".join(replicas)
    return f"{remote_clause}('{replicas_str}', {table}, '{user_name}', '{{user_password}}')"
```
</issue_to_address>

### Comment 3
<location> `ch_tools/chadmin/internal/utils.py:81` </location>
<code_context>
         )


+def get_remote_table_for_hosts(ctx: Context, table: str, replicas: list[str]) -> str:
+    """
+    Get remote/remoteSecure function for a table with given remote replicas.
</code_context>

<issue_to_address>
**suggestion (review_instructions):** Please ensure that the construction of the remote/remoteSecure table function matches ClickHouse's documented behavior, including argument order and escaping.

ClickHouse's remote/remoteSecure table functions require careful attention to argument order and escaping, especially for user/password and host lists. Please double-check that the generated string matches the documented format and handles edge cases (e.g., hostnames with special characters, empty user/password).

<details>
<summary>Review instructions:</summary>

**Path patterns:** `**/*.py`

**Instructions:**
Code that deals with ClickHouse should take into account its implementation https://github.com/ClickHouse/ClickHouse

</details>
</issue_to_address>