[GitHub] metron pull request #849: METRON-1334 Add C++11 Compliance Check to 'platfor...

2017-12-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/849 ---

[GitHub] metron pull request #851: METRON-1336 Patching Can Result in Bad Configurati...

2017-12-02 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/851#discussion_r154508609 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/ConfigurationsUtils.java --- @@ -343,25 +343,57 @@ public stat

[GitHub] metron pull request #851: METRON-1336 Patching Can Result in Bad Configurati...

2017-12-02 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/851#discussion_r154508202 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/ConfigurationsUtils.java --- @@ -343,25 +343,57 @@ public st