New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #5434: change the frequency of agent per agent #654
Fixes #5434: change the frequency of agent per agent #654
Conversation
|
||
def json = { | ||
|
||
val overrideValue = overrides.map(_.toString).getOrElse("null") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why null ?? this looks fishy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
using the null in javascript is """"""""" like """"""""" (note the quote) is like using an option type.
I use it to denote that override is not relevant ... I also could use undefined, but null is more acceted as "property has no value"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, thanks for the explaination
6f6829e
to
8394338
Compare
@@ -82,6 +86,8 @@ object DisplayNode extends Loggable { | |||
private[this] val uuidGen = RudderConfig.stringUuidGenerator | |||
private[this] val nodeInfoService = RudderConfig.nodeInfoService | |||
|
|||
private[this] val nodeRepo = RudderConfig.woNodeRepository | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why this change ? is it used somewhere ?
8394338
to
ed9c57c
Compare
ed9c57c
to
2f80453
Compare
…ey_per_node Fixes #5434: change the frequency of agent per agent
No description provided.