Only add subscription that is not an all-event sub

[EVODelavega]

Rather than sending events in their own channel (which as causing several events of the same time being queued up in their own routine, and being sent out of order), the broker now maintains a channel per event type internally, and uses that to send out events to their respective subscribers in order

[edd]

@EVODelavega is there a way that we can test that this specific change is good, or is the best option we have to verify that it fixes #2139?

[jeremyletang]

@EVODelavega is there a way that we can test that this specific change is good, or is the best option we have to verify that it fixes #2139?

For what it's worth I ran it locally and tried to get the accounts, see if ordering was respected. and it was all fine.

[vega-ci-bot]

system-tests failed.

=== Test summary ===
../../grpcTesting/governance/newAssetTests.py ....                       [100%]
../../grpcTesting/markets/marketsTests.py ..............                 [100%]
../../grpcTesting/orders/cancelOrdersTests.py ...                        [100%]
../../grpcTesting/orders/generalOrdersTests.py .............             [100%]
../../grpcTesting/orders/limitOrdersTests.py .......                     [100%]
../../grpcTesting/orders/marketOrdersTests.py ........                   [100%]
../../grpcTesting/orders/negativeOrdersTests.py ...........              [100%]
../../grpcTesting/trading/positionsTests.py .                            [100%]
../../grpcTesting/trading/tradesTests.py .......                         [100%]