I know that right now `admin` has user id 1, but in the future it could change 
to be 0 and it would break this code. I think it's better to be more explicit 
here an unravel the one liner:

```python
client_id = request.form.get('client_id') or utils.shortid()

try:
    user_id = int(g.user.get_id())
except Exception:
    user_id = None

query = Query(
    ...
    user_id=user_id,
    client_id=client_id,
)
```

[ Full content available at: 
https://github.com/apache/incubator-superset/pull/5730 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to