metron-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mmiklavc <>
Subject [GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack
Date Mon, 02 Apr 2018 16:45:32 GMT
Github user mmiklavc commented on a diff in the pull request:
    --- Diff: metron-interface/metron-rest/src/main/scripts/ ---
    @@ -36,6 +36,7 @@ METRON_SYSCONFIG="${METRON_SYSCONFIG:-/etc/default/metron}"
    --- End diff --
    @nickwallen This is consistent with what we've done for the parsers (see line 38 immediately
above). We could possibly refactor, but I wouldn't advise it as part of this PR. Not defining
a default would mean the responsibility is now on the end user for indexing, but not for parsers.
I think that is going to be more confusing to a user in the current state. I do agree that
these could probably be moved to the defaults script as a follow-on refactoring PR.


View raw message