Replace usages of TPL DataFlow BufferBlock with Threading.Channels

[eerhardt]

There are 2 places we are using BufferBlock<T> today:

machinelearning/src/Microsoft.ML.Data/Transforms/RowShufflingTransformer.cs

Lines 486 to 487 in b7db4fa

	private readonly BufferBlock<int> _toProduce;
	private readonly BufferBlock<int> _toConsume;

machinelearning/src/Microsoft.ML.Sweeper/AsyncSweeper.cs

Line 171 in b7db4fa

private readonly BufferBlock<ParameterSetWithId> _paramQueue;

We should consider replacing this dependency with https://www.nuget.org/packages/System.Threading.Channels/ instead. Channels are a bit simpler, and more performant than BufferBlock.

See:

• Ensure BufferBlocks are completed and empty in RowShufflingTransformer. #4479 (comment)
• https://github.com/dotnet/corefx/issues/24715

[mariombaltazar]

I, Can I try do this?

[eerhardt]

@mariombaltazar -Yes! Feel free.

[mariombaltazar]

I @eerhardt,
I can not do pull request. can you help me?

[eerhardt]

Here's the steps to create a pull request:

1. Fork the repo - https://help.github.com/en/github/getting-started-with-github/fork-a-repo
2. Push your changes to that fork
3. Create the PR from your fork - https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request-from-a-fork

[mstfbl]

Hi @mariombaltazar , are you still interested in working on this issue?

[campiador]

@eerhardt @mstfbl Hi! In case this issue is available, may I please start working on it?

[antoniovs1029]

Hi @campiador . It is still an unsolved issue, and it seems that @mariombaltazar isn't working on it, so you can go for it 😃

[frank-dong-ms-zz]

Close this issue and this already been addressed.