streams icon indicating copy to clipboard operation
streams copied to clipboard
verified publisher icon palantir

Delete deprecated+buggy methods in MoreStreams

Open ash211 opened this issue 2 years ago • 0 comments

In https://github.com/palantir/streams/pull/36 we marked two methods as deprecated:

  • MoreStreams#inCompletionOrder (the overload without an Executor)
  • MoreStreams#blockingStreamWithParallelism (the overload without an Executor)

Uses of those two methods still remain in our codebase and are causing issues (ref internal foundry/interventions#4222).

These methods have been marked deprecated for almost a year and a half. Can we delete them? There are 51+ potential usages (see internal https://pl.ntr/2hY)

ash211 avatar Aug 28 '23 22:08 ash211