Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Beta from ListeningScheduledExecutorService? #3340

Closed
pkwarren opened this issue Dec 17, 2018 · 0 comments · Fixed by #3393
Closed

Remove Beta from ListeningScheduledExecutorService? #3340

pkwarren opened this issue Dec 17, 2018 · 0 comments · Fixed by #3393
Assignees
Labels
package=concurrent status=fixed type=debeta Request to remove something from @Beta

Comments

@pkwarren
Copy link

The MoreExecutors#listeningDecorator(ScheduledExecutorService delegate) method is not marked Beta but returns a ListeningScheduledExecutorService which is marked Beta. Is it planned to remove the Beta designation from the ListeningScheduledExecutorService interface in the future?

Today (in order to avoid errors about using beta APIs), consumers must downcast to a ListeningExecutorService or ScheduledExecutorService in order to use the returned result from this method.

@kluever kluever added package=concurrent type=debeta Request to remove something from @Beta labels Dec 26, 2018
@ronshapiro ronshapiro mentioned this issue Feb 14, 2019
ronshapiro pushed a commit that referenced this issue Feb 14, 2019
collect
- most Collectors (aside from BloomFilter and ImmutableRange*, as those whole classes are @beta (though I did remove the redundant @beta annotation from the Collector-returning methods for clarity))
- Maps: asConverter, immutableEnumMap
- MultimapBuilder
- Streams: stream(Iterable), concat

io
- ByteStreams: copy, toByteArray
- CharStreams: copy, toString
- Files: as(Char|Byte)(Source|Sink)

util.concurrent
- Futures: addCallback, getDone, getUnchecked, immediateCancelledFuture, immediateFailedFuture, immediateFuture
- ListeningScheduledExecutorService
- Uninterruptibles

other
- HtmlEscapers
- Splitter.splitToList
- Ticker

Fixes #3287
Fixes #3251 (aside from ImmutableRange*, but those whole classes are @beta, anyway)
Addresses the main concerns of #3285 but doesn't cover the broader request
Fixes #3340
Partially addresses #3239

RELNOTES=Removed `@Beta` from a number of frequently used APIs.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=232681253
@ronshapiro ronshapiro mentioned this issue Feb 26, 2019
ronshapiro pushed a commit that referenced this issue Feb 26, 2019
collect
- most Collectors (aside from BloomFilter and ImmutableRange*, as those whole classes are @beta (though I did remove the redundant @beta annotation from the Collector-returning methods for clarity))
- Maps: asConverter, immutableEnumMap
- MultimapBuilder
- Streams: stream(Iterable), concat

io
- ByteStreams: copy, toByteArray
- CharStreams: copy, toString
- Files: as(Char|Byte)(Source|Sink)

util.concurrent
- Futures: addCallback, getDone, getUnchecked, immediateCancelledFuture, immediateFailedFuture, immediateFuture
- ListeningScheduledExecutorService
- Uninterruptibles

other
- HtmlEscapers
- Splitter.splitToList
- Ticker

Fixes #3287
Fixes #3251 (aside from ImmutableRange*, but those whole classes are @beta, anyway)
Addresses the main concerns of #3285 but doesn't cover the broader request
Fixes #3340
Partially addresses #3239

RELNOTES=Removed `@Beta` from a number of frequently used APIs.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=232681253
ronshapiro pushed a commit that referenced this issue Feb 26, 2019
collect
- most Collectors (aside from BloomFilter and ImmutableRange*, as those whole classes are @beta (though I did remove the redundant @beta annotation from the Collector-returning methods for clarity))
- Maps: asConverter, immutableEnumMap
- MultimapBuilder
- Streams: stream(Iterable), concat

io
- ByteStreams: copy, toByteArray
- CharStreams: copy, toString
- Files: as(Char|Byte)(Source|Sink)

util.concurrent
- Futures: addCallback, getDone, getUnchecked, immediateCancelledFuture, immediateFailedFuture, immediateFuture
- ListeningScheduledExecutorService
- Uninterruptibles

other
- HtmlEscapers
- Splitter.splitToList
- Ticker

Fixes #3287
Fixes #3251 (aside from ImmutableRange*, but those whole classes are @beta, anyway)
Addresses the main concerns of #3285 but doesn't cover the broader request
Fixes #3340
Partially addresses #3239

RELNOTES=Removed `@Beta` from a number of frequently used APIs.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=232681253
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package=concurrent status=fixed type=debeta Request to remove something from @Beta
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants