Sanskar Modi

Results 40 comments of Sanskar Modi

@SteNicholas Added. Q: do you want me to make use of it in this PR?

> the implementation of ConfigService could use getName interface to load the implementation instance instead of hard code in factory. @SteNicholas Trying to understand this comment better – I think...

@SteNicholas @waitinfuture @pan3793 wdyt about this change. We can fix this at more places incrementally.

@FMX this mainly aims to improve the Master and Worker logging. If i want to keep such logs but don't want it in human readable way to avoid new lines.

I've made tags part of SystemConfig as it made more sense to me, let me know if you think if it should extracted out as a separate config something like...

@SteNicholas We're currently not in production but this will help us tune the forceExitTimeout config better and see if the default value is working for us or not. As we...

Sorry messed up the commit history on this one, reopening the PR again. @SteNicholas updated the PR with dashboard changes.