You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Follow-up from #140: that PR adds a log.Info on the default case (Enabled=false for Cassandra and ES), while it doesn't let the user know that the dependency job isn't installed when Enabled=true and storage isn't Cassandra or ES.
It should:
Not log anything on the standard, common case. If everything is as it should, there's no need to log anything
Log (debug or info) when Enabled=true and the storage isn't ES or Cassandra. This is because the flag is false by default, meaning that the user has explicitly set the value to true, so, there's a clear expectation that the job will be installed.
The text was updated successfully, but these errors were encountered:
Follow-up from #140: that PR adds a
log.Info
on the default case (Enabled=false for Cassandra and ES), while it doesn't let the user know that the dependency job isn't installed when Enabled=true and storage isn't Cassandra or ES.It should:
false
by default, meaning that the user has explicitly set the value totrue
, so, there's a clear expectation that the job will be installed.The text was updated successfully, but these errors were encountered: