vectordotdev / vector

A high-performance observability data pipeline.
https://vector.dev
Mozilla Public License 2.0
17.36k stars 1.51k forks source link

chore: Add comments/note deprecated sink code #12932

Closed spencergilbert closed 1 year ago

spencergilbert commented 2 years ago

We've updated our sinks to use Stream rather than Sink, but we haven't made it obvious the old code shouldn't be used anymore.

We don't necessarily want to use #[deprecated] as we don't necessarily want to raise those warnings right now.

StephenWakely commented 2 years ago

This is a list of our sinks and those that are using deprecated components.

| Sink                    | New style                                   |
|-------------------------+---------------------------------------------|
| clickhouse              | BatchedHttpSink                             |
| influxdb_metrics        | BatchSink                                   |
| redis                   | BatchSink                                   |
| aws_cloudwatch_logs     | yes                                         |
| console                 | yes                                         |
| gcp_pubsub              | BatchedHttpSink                             |
| kafka                   | yes                                         |
| aws_cloudwatch_metrics  | PartitionBatchSink                          |
| datadog_archives        | yes                                         |
| gcp_stackdriver_logs    | BatchedHttpSink                             |
| logdna                  | PartitionHttpSink                           |
| sematext_logs           | wraps elasticsearch                         |
| aws_kinesis_firehose    | yes                                         |
| gcp_stackdriver_metrics | BatchedHttpSink                             |
| loki                    | yes                                         |
| sematext_metrics        | yes                                         |
| aws_kinesis_streams     | yes                                         |
| datadog_events          | yes                                         |
| honeycomb               | BatchedHttpSink                             |
| nats                    | yes                                         |
| socket                  | yes                                         |
| aws_s3                  | yes                                         |
| datadog_logs            | yes                                         |
| http                    | BatchedHttpSink                             |
| new_relic               | yes                                         |
| splunk_hec_logs         | yes                                         |
| aws_sqs                 | yes                                         |
| datadog_metrics         | yes                                         |
| humio_logs              | yes - wraps splunk_hec_logs                 |
| new_relic_logs          | wraps HttpSink                              |
| axiom                   | wraps ElasticSearch                         |
| datadog_traces          | yes                                         |
| papertrail              | wraps TcpSink                               |
| splunk_hec_metrics      | yes                                         |
| azure_blob              | yes                                         |
| elasticsearch           | yes                                         |
| humio_metrics           | wraps HumioLogs which wraps splunk_hec_logs |
| prometheus_exporter     | yes                                         |
| statsd                  | Wraps TcpSink, BatchSink or UnixSink        |
| azure_monitor_logs      | BatchedHttpSink                             |
| file                    | yes                                         |
| influxdb_logs           | BatchedHttpSink                             |
| prometheus_remote_write | PartitionBatchSink                          |
| vector  v1              | wraps TcpSink                               |
| vector  v2              | yes                                         |
| blackhole               | yes                                         |
| gcp_cloud_storage       | yes                                         |
spencergilbert commented 2 years ago

Should be able to cross reference with: https://github.com/vectordotdev/vector/issues/9261

jszwedko commented 1 year ago

Closed by https://github.com/vectordotdev/vector/pull/13626