refactor: Remove uncommented concurrent batch processor and make note…

… in readme
lightstep · Jul 1, 2024 · 3b9db5b · 3b9db5b
1 parent a68e9ee
commit 3b9db5b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/arrow/otelcolarrow-build.yaml b/arrow/otelcolarrow-build.yaml
@@ -100,8 +100,8 @@ receivers:
 
 processors:
   # We recommend use of the OTel-Arrow concurrent batch processor.
-  #  gomod: github.com/open-telemetry/otel-arrow/collector v0.24.0
-  #  import: github.com/open-telemetry/otel-arrow/collector/processor/concurrentbatchprocessor
+  - gomod: github.com/open-telemetry/otel-arrow/collector v0.24.0
+    import: github.com/open-telemetry/otel-arrow/collector/processor/concurrentbatchprocessor
 
   # We recommend building with the follow processor for obfuscation, in case you want to
   # record telemetry sessions for offline analysis.
@@ -112,7 +112,7 @@ processors:
   # processor or the core batch processor, i.e., do not build with
   # either of:
   # - go.opentelemetry.io/collector/processor/memorylimiterprocessor.
-  - gomod: go.opentelemetry.io/collector/processor/batchprocessor v0.103.0
+  # - gomod: go.opentelemetry.io/collector/processor/batchprocessor v0.103.0
 
   # These Collector-Contrib components are referred to in the charts in this package.
   - gomod: github.com/open-telemetry/opentelemetry-collector-contrib/processor/resourcedetectionprocessor v0.103.0

diff --git a/example/README.md b/example/README.md
@@ -46,3 +46,9 @@ builder --config ./otelcolarrow-build.yaml
 ```bash
 ./dist/otelarrowcol --config ../example/vm/config.yaml
 ```
+
+#### Troubleshooting
+
+- Note this example config does not use the concurrent batch processor. Uncomment this line instead:
+
+` - gomod: go.opentelemetry.io/collector/processor/batchprocessor v0.103.0`