Revert "Add SocketDataTransport to support kTCP/kUDP (#540)"

[ThomsonTan]

This reverts commit 2184b83.

There is no plan to support kTCP/kUDP in the near future. Will log error when these protocols are used.

FYI @zurex

[malkia]

Can anyone explain what happened here? This was added first, then removed, but no further notes about why?

[lalitb]

Can anyone explain what happened here? This was added first, then removed, but no further notes about why?

The exporter in this directory is designed only for sending metrics to a locally running collector agent - so it supports UDS on Linux and ETW on Windows. For sending metrics to remote collector, it’s best to use the standard OTLP exporter.

[ThomsonTan]

Can anyone explain what happened here? This was added first, then removed, but no further notes about why?

Yes, as Lalit mentioned, we recommend running this exporter side-by-side with the application. To reduce the confusion, and as mentioned in the revert description, we don't plan to support the remote protocol like TCP or UDP.

I will remove the protocol definition of kTCP and 'kUDP` to make this more clear.