Remove old reviewer notes (#6655)

elastic · Mar 27, 2018 · 867c23c · 867c23c
1 parent 1f8d072
commit 867c23c
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 34 deletions.
diff --git a/filebeat/docs/inputs/input-common-harvester-options.asciidoc b/filebeat/docs/inputs/input-common-harvester-options.asciidoc
@@ -113,10 +113,6 @@ decoding with filtering and multiline if you set the `message_key` option. This
 can be helpful in situations where the application logs are wrapped in JSON
 objects, as with like it happens for example with Docker.
 
-// REVIEWERS: Is this setting required/necessary when the docker input is used?
-// If so, what should I say? I'll need to conditionally code anything related
-// to docker because this description shows up in multiple input types.
-
 Example configuration:
 
 [source,yaml]

diff --git a/filebeat/docs/modules/kafka.asciidoc b/filebeat/docs/modules/kafka.asciidoc
@@ -45,11 +45,6 @@ file to override the default paths for logs:
 -----
 
 
-// REVIEWERS: I must be doing something wrong with the config settings. The
-// above config works, but when I try to specify var.kafka_home, it doesn't
-// seem to have any effect. 
-
-
 To specify the same settings at the command line, you use:
 
 ["source","sh",subs="attributes"]
@@ -66,14 +61,6 @@ include::../include/config-option-intro.asciidoc[]
 [float]
 ==== `log` fileset settings
 
-// REVIEWERS: I've added a description because this variable appears in the
-// kafka.yml file. However, I don't understand how it works.
-
-//*`var.kafka_home`*::
-
-//The home path for Kafka. If this variable is not set,  {beatname_uc} looks under
-//`/opt`.
-
 include::../include/var-paths.asciidoc[]
 
 

diff --git a/filebeat/docs/modules/traefik.asciidoc b/filebeat/docs/modules/traefik.asciidoc
@@ -19,8 +19,6 @@ This module requires the
 {elasticsearch-plugins}/ingest-user-agent.html[ingest-user-agent] and
 {elasticsearch-plugins}/ingest-geoip.html[ingest-geoip] Elasticsearch plugins.
 
-//REVIEWERS: Do we need to say anything else about compatibility here?
-
 include::../include/running-modules.asciidoc[]
 
 [float]

diff --git a/filebeat/module/kafka/_meta/docs.asciidoc b/filebeat/module/kafka/_meta/docs.asciidoc
@@ -40,11 +40,6 @@ file to override the default paths for logs:
 -----
 
 
-// REVIEWERS: I must be doing something wrong with the config settings. The
-// above config works, but when I try to specify var.kafka_home, it doesn't
-// seem to have any effect. 
-
-
 To specify the same settings at the command line, you use:
 
 ["source","sh",subs="attributes"]
@@ -61,12 +56,4 @@ include::../include/config-option-intro.asciidoc[]
 [float]
 ==== `log` fileset settings
 
-// REVIEWERS: I've added a description because this variable appears in the
-// kafka.yml file. However, I don't understand how it works.
-
-//*`var.kafka_home`*::
-
-//The home path for Kafka. If this variable is not set,  {beatname_uc} looks under
-//`/opt`.
-
 include::../include/var-paths.asciidoc[]
diff --git a/filebeat/module/traefik/_meta/docs.asciidoc b/filebeat/module/traefik/_meta/docs.asciidoc
@@ -14,8 +14,6 @@ This module requires the
 {elasticsearch-plugins}/ingest-user-agent.html[ingest-user-agent] and
 {elasticsearch-plugins}/ingest-geoip.html[ingest-geoip] Elasticsearch plugins.
 
-//REVIEWERS: Do we need to say anything else about compatibility here?
-
 include::../include/running-modules.asciidoc[]
 
 [float]