This is a bit hard to read — at first I thought these were 3 arguments being
passed, and `get` was a function.
Personally I would rewrite this as:
```python
extra = self.get_extra()
try:
cache_timeout = extra['metadata_cache_timeout']['schema_cache_timeout']
except KeyError:
cache_timeout = None
```
There's [a proposal](https://www.python.org/dev/peps/pep-0505/) that would
allow this to be written as:
```python
cache_timeout =
self.get_extra()?['metadata_cache_timeout']?['schema_cache_timeout']
```
[ Full content available at:
https://github.com/apache/incubator-superset/pull/5933 ]
This message was relayed via gitbox.apache.org for [email protected]
