https://github.com/osquery/osquery logo
Title
m

marpaia

10/26/2018, 5:28 PM
@gguli @fmanco i noticed this PR that recently removed an alias to a flag which enabled the event format for differential query logs: https://github.com/facebook/osquery/pull/4970 the motivation of the PR is said to simplify the logger, but is one LoC removed worth causing an API change? i don’t think there’s a huge problem with leaving aliases for as long as possible, but if we are introducing potentially unnecessary api changes, they shouldn’t be patch releases. also, the event format is awesome, i just wanted to chime in that, at work, i use it exclusively for our logging pipeline because of some distinct consistency guarantees you can provide with it vs the default format.
f

fmanco

10/29/2018, 9:22 PM
Hi, I think the intent was to start cleaning the interface, not to remove features. We won't remove any features in the near future and we will submit blueprints if we intend to do so.
m

marpaia

10/30/2018, 5:37 PM
again, one LoC removed isn’t worth the breaking API change.
and you introduced a breaking API change in a patch release…