Remove (seemingly??) unused pkg/buses

[vaikas]

Addresses #591

Proposed Changes

• Remove unused files from /pkg/buses

[knative-prow-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vaikas-google

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [vaikas-google]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[neosab]

Fanout sidecar uses pkg/bus Message Dispatcher

eventing/pkg/sidecar/fanout/fanout_handler.go

Line 123 in 8589a29

return f.dispatcher.DispatchMessage(&m, sub.SubscriberURI, sub.ReplyURI, buses.DispatchDefaults{})

Kafka Dispatcher uses the pkg/busMessage Dispatcher & Receiver #589

Ideally, they should be moved to proper utility packages rather than leveraging parts of the old bus implementation.

[neosab]

Ah, my bad didn't see this PR was closed.

[knative-prow-robot]

@vaikas-google: The following tests failed, say /retest to rerun them all:

Test name	Commit	Details	Rerun command
pull-knative-eventing-unit-tests	ab5b06d	link	/test pull-knative-eventing-unit-tests
pull-knative-eventing-build-tests	ab5b06d	link	/test pull-knative-eventing-build-tests
pull-knative-eventing-integration-tests	ab5b06d	link	/test pull-knative-eventing-integration-tests

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[matzew]

I think the package could be named differently. I was wondering exactly the same, than noticed that it's actually used ;-)