Skip to content
This repository has been archived by the owner on Feb 3, 2021. It is now read-only.

Fix: list_applications throwing error #692

Merged
merged 3 commits into from
Jan 23, 2019
Merged

Conversation

brettsimons
Copy link
Contributor

list_applications was previously throwing this error:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "...\aztk\spark\client\base\operations.py", line 85, in list_applications
    return list_applications.list_applications(self, id)
  File "...\aztk\spark\client\base\helpers\list_applications.py", line 21, in list_applications
    return models.Application(_list_applications(core_operations, id))
  File "...\aztk\spark\client\base\helpers\list_applications.py", line 11, in _list_applications
    scheduling_target = core_operations.get_cluster_configuration(id).scheduling_target
AttributeError: 'ClusterOperations' object has no attribute 'get_cluster_configuration'

It is now working and returns a list of tasks.

@brettsimons
Copy link
Contributor Author

Here is a workaround for the current release: Client.cluster._core_cluster_operations.list_batch_tasks(cluster_id)

@jafreck
Copy link
Member

jafreck commented Jan 9, 2019

Thanks for reporting this, can you confirm, were you experiencing this issue on the latest released version (0.10.2)?

@brettsimons
Copy link
Contributor Author

Yes, I've only validated that this is an issue in 0.10.2. After looking a bit more at the source, I'll update the fix to return an array of aztk.spark.models.Application.

…nstead of the raw tasks as would be expected by the name. Also updated documentation to reflect this change.
Copy link
Member

@jafreck jafreck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great -- thanks for fixing!

@jafreck jafreck merged commit 72c7833 into Azure:master Jan 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants