Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r92495353
--- Diff:
metron-analytics/metron-profiler/src/main/config/profiler.properties ---
@@ -18,21 +18,44 @@
#
#
-##### Storm #####
+##### Profiler #####
-profiler.workers=1
-profiler.executors=0
+# topic from which the profiler consumes messages
profiler.input.topic=indexing
+
+# how frequently the profiler summarizes data - aka duration of each
profile period
profiler.period.duration=15
profiler.period.duration.units=MINUTES
+
+# lifespan of a profile - must be greater than twice the profile period
+# intermediate state may be lost if no messages received within this time
span
profiler.ttl=30
profiler.ttl.units=MINUTES
-profiler.hbase.salt.divisor=1000
+
+# event time processing - the name of the field containing the event
timestamp
+profiler.event.timestamp.field=timestamp
+
+# event time processing - how long time lags behind the last seen timestamp
+profiler.event.time.lag=1
--- End diff --
That wasn't a clear comment on my part, can we please update the readme to
understand exactly the impact of that parameter. As it stands, I'm not sure
about the tradeoff for making it 0 minutes, for instance.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
