[Bug]: Wrong typehint for describe_role

[graelo]

Is there an existing issue for this?

• I have searched the existing issues

Describe the bug

When calling client.describe_role(role_name=some_role), the function returns a dictionary, and this is pretty clear from the code:

pymilvus/pymilvus/milvus_client/milvus_client.py

Lines 967 to 979 in 650f7cd

	def describe_role(
	self, role_name: str, timeout: Optional[float] = None, **kwargs
	) -> List[Dict]:
	conn = self._get_connection()
	db_name = kwargs.pop("db_name", "")
	try:
	res = conn.select_grant_for_one_role(role_name, db_name, timeout=timeout, **kwargs)
	except Exception as ex:
	raise ex from ex
	ret = {}
	ret["role"] = role_name
	ret["privileges"] = [dict(i) for i in res.groups]
	return ret

However, the function signature indicates the returned value is List[Dict].

Expected Behavior

The function signature should simply indicate the returned value is Dict, or Dict[str, Any].

Steps/Code To Reproduce behavior

I believe this is unnecessary for this type of issue.

Environment details

Irrelevant.

Anything else?

I know this seems minor, but with type checkers like pyright, I assure you this kind of problem is a bit confusing.

[XuanYang-cn]

/assign @CaoHaiNam

[sre-ci-robot]

@XuanYang-cn: GitHub didn't allow me to assign the following users: CaoHaiNam.

Note that only milvus-io members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

/assign @CaoHaiNam

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[XuanYang-cn]

/close

[sre-ci-robot]

@XuanYang-cn: Closing this issue.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.