Consolidate env reading to single config object.#600
Conversation
fa21286 to
95cb0d6
Compare
![datadog-datadog-prod-us1[bot]](https://avatars.githubusercontent.com/u/365230?s=60&v=4){kind=small}
| return val.lower() == "true" or val == "1" | ||
|
|
||
|
|
||
| class Config: |
There was a problem hiding this comment.
⚪ Code Quality Violation
Class Config should have an init method (...read more)
Ensure that a class has an __init__ method. This check is bypassed when the class is a data class (annotated with @dataclass).
7d2950e to
dd6b1d2
Compare
| # int. let's special case that here, because it's sure to happen in | ||
| # customer code. | ||
| err = d.get("error") | ||
| if err and type(err) == bool: |
There was a problem hiding this comment.
⚪ Code Quality Violation
use isinstance() instead of type() (...read more)
Using type checks if an object o is exactly a given type. In contrast, isinstance checks if o is a given type, or any subclass of the given type, thus supporting inheritance. In addition,isinstance is generally faster than type.
37e4ccb to
6449589
Compare
6449589 to
2957287
Compare
| end_time_ns = time.time_ns() | ||
| node.end_time_ns = end_time_ns | ||
| if not import_stack: # import_stack empty, a root node has been found | ||
| global root_nodes |
There was a problem hiding this comment.
These globals are not required because the affected objects are lists.
f869908 to
7bc8751
Compare
| return [val.strip() for val in val.split(",") if val.strip()] | ||
|
|
||
|
|
||
| class Config: |
There was a problem hiding this comment.
⚪ Code Quality Violation
Class Config should have an init method (...read more)
Ensure that a class has an __init__ method. This check is bypassed when the class is a data class (annotated with @dataclass).
joeyzhao2018
left a comment
joeyzhao2018
left a comment
There was a problem hiding this comment.
LGTM. Thank you for doing this.
3 participants
What does this PR do?
Consolidate all config and (most) env var reading into a single
configobject.Furthermore, the environment is now only read when it's needed. The config object will only resolve a config setting when it is accessed. Meaning, we'll be reading a ton fewer environment variables during initialization which will lead to faster cold starts.
Motivation
Tests are currently failing on #598 with a circular import error. Consolidating all config reading to a single location will make fixing that much much much easier.
Testing Guidelines
Additional Notes
Note that for all the boolean like env vars, it is now possible to set the environment value to
"1"to configure for aTrueevaluation. The customer can then use"0"or really anything other than"true"or"1"to get aFalseevaluation.Types of Changes
Check all that apply