----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51504/#review147384 -----------------------------------------------------------
flume-ng-core/src/main/java/org/apache/flume/formatter/output/BucketPath.java (line 44) <https://reviews.apache.org/r/51504/#comment214516> Why include "." and "-" in the group 3 pattern? Also, I don't think % is regex syntax. In that case, group 3 can be simply: %\[(\w+)\] flume-ng-core/src/main/java/org/apache/flume/formatter/output/BucketPath.java (line 223) <https://reviews.apache.org/r/51504/#comment214514> how about key.toLowerCase() flume-ng-core/src/main/java/org/apache/flume/formatter/output/BucketPath.java (line 453) <https://reviews.apache.org/r/51504/#comment214515> Please add comment: Group 3 is the %[...] pattern. flume-ng-doc/sphinx/FlumeUserGuide.rst (line 1781) <https://reviews.apache.org/r/51504/#comment214517> typo: Substitute flume-ng-doc/sphinx/FlumeUserGuide.rst (line 1784) <https://reviews.apache.org/r/51504/#comment214518> "which may fail in some networking environments" Also, please wrap non-table text at 100 chars - Mike Percy On Aug. 30, 2016, 3:29 p.m., Jeff Holoman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51504/ > ----------------------------------------------------------- > > (Updated Aug. 30, 2016, 3:29 p.m.) > > > Review request for Flume and Grant Henke. > > > Repository: flume-git > > > Description > ------- > > Added static escape strings localhost, IP and FQDN > > > Diffs > ----- > > > flume-ng-core/src/main/java/org/apache/flume/formatter/output/BucketPath.java > f640ec9791fb6e067c20ef797691a617a8e472d6 > > flume-ng-core/src/test/java/org/apache/flume/formatter/output/TestBucketPath.java > ccc746066e420e772021a4bc758a81736225f76d > flume-ng-doc/sphinx/FlumeUserGuide.rst > 7e207aa825f56ea32bd5282e75ef02090b21c3f7 > > Diff: https://reviews.apache.org/r/51504/diff/ > > > Testing > ------- > > > Thanks, > > Jeff Holoman > >
