[GitHub] metron pull request #1146: METRON-1704 Message Timestamp Logic Should be Sha...

[nickwallen]

Github user nickwallen commented on a diff in the pull request:

    https://github.com/apache/metron/pull/1146#discussion_r208915112
  
    --- Diff: 
metron-analytics/metron-profiler/src/main/java/org/apache/metron/profiler/bolt/ProfileSplitterBolt.java
 ---
    @@ -162,15 +156,9 @@ private void doExecute(Tuple input) throws 
ParseException, UnsupportedEncodingEx
         // ensure there is a valid profiler configuration
         ProfilerConfig config = getProfilerConfig();
         if(config != null && config.getProfiles().size() > 0) {
    +      routeMessage(input, message, config);
     
    -      // what time is it?
    --- End diff --
    
    Here I removed the timestamp logic from Storm.  The `MessageRouter` does 
this for us now.


---