astropy / astropy/pyvo

`AsyncTAPJob` makes unnecessary network calls in property getter.

Open
#365 3 comments 0 reactions 0 assignees View on GitHub
bug component: TAP
Dominant language
Python
Stars
88
Forks
64
Avg merge
38m
Merged PRs (30d)
1

Description

It looks like most of not all the mutable properties of the class `AsynTAPJob` make network calls to update the job info. Most of the time this seems an unnecessary overhead. At the very minimum, the code could be changed to avoid the `_update` calls if the job is in a final state, in which case the remote call is redundant.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the AsyncTAPJob class and reviewing its mutable property getters and _update calls, then inspect any existing tests for job state and remote updates. Done means final-state getters avoid redundant network calls while non-final jobs retain their current update behavior, with tests covering both cases.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.