[ISSUE] Recursive workspace listing

[mattfysh]

Description
A recursive kwarg is documented for workspace.list, but does not exist: https://databricks-sdk-py.readthedocs.io/en/latest/workspace/workspace.html

Reproduction
Testing exists for it:

databricks-sdk-py/tests/integration/test_workspace.py

Lines 6 to 10 in 3a2d2e6

	def test_workspace_recursive_list(w, random):
	names = []
	for i in w.workspace.list(f'/Users/{w.current_user.me().user_name}', recursive=True):
	names.append(i.path)
	assert len(names) > 0

But it does not exist in the source code:

databricks-sdk-py/databricks/sdk/service/workspace.py

Lines 1416 to 1442 in 3a2d2e6

	def list(self,
	path: str,
	*,
	notebooks_modified_after: Optional[int] = None,
	**kwargs) -> Iterator[ObjectInfo]:
	"""List contents.
	Lists the contents of a directory, or the object if it is not a directory. If the input path does not
	exist, this call returns an error `RESOURCE_DOES_NOT_EXIST`.
	:param path: str
	The absolute path of the notebook or directory.
	:param notebooks_modified_after: int (optional)
	UTC timestamp in milliseconds
	:returns: Iterator over :class:`ObjectInfo`
	"""
	request = kwargs.get('request', None)
	if not request: # request is not given through keyed args
	request = ListWorkspaceRequest(notebooks_modified_after=notebooks_modified_after, path=path)
	query = {}
	if notebooks_modified_after: query['notebooks_modified_after'] = request.notebooks_modified_after
	if path: query['path'] = request.path
	json = self._api.do('GET', '/api/2.0/workspace/list', query=query)
	return [ObjectInfo.from_dict(v) for v in json.get('objects', [])]

Expected behavior
Recursively list notebooks at the given path

[nfx]

@mattfysh it's defined in https://github.com/databricks/databricks-sdk-py/blob/v0.1.12/databricks/sdk/mixins/workspace.py#L15-L36 - e.g. if something is integration tested - then it definitely works ;)

this is a bug in https://github.com/databricks/databricks-sdk-py/blob/main/docs/gen-client-docs.py, thanks for pointing it out.

[neilbest-db]

Perhaps this is resolved by #193?

[mgyucht]

I think this is just a misunderstanding: the parameter does exist, but the WorkspaceExt replaces the implementation in WorkspaceAPI for users. There was a recent refactoring to the docs pages that does include WorkspaceExt as well as our other mixins, so it should match the SDK exactly.