Nope. The `state` is still `running` and that's why the inconsistency between
progress bar and status happens.
So after query succeeds, the `action.results` is returned from the server
containing something like
```
{
'query': {
'id': ...,
'state': 'success',
'progress': 100,
...
}
'status': 'success'
}
```
but `action.query` itself is a part of the state of the app which contains
`{'state': 'running'}` field in itself.
Then in the next round of rerendering, the query data returned from the server
will show in the sqllab `SouthPane` and `state` will be updated in
`QueryAutoRefresh` from `running` to `success`:
https://github.com/apache/incubator-superset/blob/cb92739c46355c3217b33297cbcce8801f5768aa/superset/assets/src/SqlLab/components/QueryAutoRefresh.jsx#L45
So the inconsistency only exists during the period of rerendering, but users
are still able to to see it and get confused.
Please correct me if my understanding is wrong.
[ Full content available at:
https://github.com/apache/incubator-superset/pull/5848 ]
This message was relayed via gitbox.apache.org for [email protected]
