[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_r208305293
  
    --- Diff: 
metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/StandAloneProfiler.java
 ---
    @@ -112,26 +112,14 @@ public StandAloneProfiler(ProfilerConfig config,
        * @param message The message to apply.
        */
       public void apply(JSONObject message) {
    -
    -    // what time is it?
    -    Clock clock = clockFactory.createClock(config);
    -    Optional<Long> timestamp = clock.currentTimeMillis(message);
    -
    -    // can only route the message, if we have a timestamp
    -    if(timestamp.isPresent()) {
    --- End diff --
    
    Here I removed the timestamp logic from the REPL.  The `MessageRouter` does 
this for us now.


---