guava icon indicating copy to clipboard operation
guava copied to clipboard

Remove @Beta from RangeSet and RangeMap

Open tyomka opened this issue 6 years ago • 11 comments

Range has been marked with @Beta since 10.0 (September 28, 2011). RangeMap, RangeSet and their implementations since 14.0 (February 25, 2013).

I would like to propose to remove the @Beta annotation.

tyomka avatar Feb 06 '19 11:02 tyomka

(The only part of Range that's still beta is asSet().) I feel like we should be able to take care of these in the next batch of debeta-ings.

kevinb9n avatar Feb 06 '19 23:02 kevinb9n

@cpovirk I've mistakenly assumed that Range is annotated with @Beta. It is not. Still the issue context is still relevant for RangeMap and RangeSet.

tyomka avatar Feb 08 '19 14:02 tyomka

@tyomka Can I work on this?

shashvat-kedia avatar Jul 31 '19 10:07 shashvat-kedia

Is there any update on this?

gskaplan avatar Jan 17 '20 23:01 gskaplan

Is there any news?

prissky avatar May 13 '20 11:05 prissky

Any news about this?

ggarcia-te avatar Jun 29 '20 19:06 ggarcia-te

Any news about this?

I'm kinda waiting for someone from the Guava team to validate my pull request. Is there anything i can do?

tyomka avatar Jun 29 '20 21:06 tyomka

+1 to remove @Beta. Apache Calcite, as of release 1.26, relies on RangeSet (and it appears in the Calcite's public API in one place, class Sarg). We are prepared to shade, and break our API if it comes to that, because RangeSet is the perfect library for what we need. Some certainty would be appreciated. If RangeSet is going away we can make other plans.

julianhyde avatar Sep 29 '20 19:09 julianhyde

I can see that the issue is open for more than a year. Is the Guava team at least considering this request?

zabetak avatar Sep 30 '20 10:09 zabetak

Any updates on removing @Beta for RangeMap (also applies to TreeRangeMap)?

mwunderlich avatar Jun 02 '21 06:06 mwunderlich

Any updates on removing @beta for RangeMap (also applies to TreeRangeMap)?

runnable-thread avatar Jan 18 '22 00:01 runnable-thread

This is now fixed.

kluever avatar Apr 10 '23 18:04 kluever

This is now fixed.

@kluever Do you know which version this fix is going to be released in? Latest version 31.1-jre doesn't have it

taomoh avatar May 17 '23 17:05 taomoh

It'll be in the next version of Guava --- I think @cpovirk was planning on cutting a release in the next few weeks?

kluever avatar May 17 '23 21:05 kluever

Yes, maybe the end of next week if all goes well. If not, I'd hope the week after.

cpovirk avatar May 18 '23 14:05 cpovirk

@cpovirk, It's helpful if you can state the release number when it will be fixed. Because many of us dependent issues and we want to write 'Fixed in Guava 32.0' and move on.

julianhyde avatar May 18 '23 16:05 julianhyde

It will be 32.0.0. (Even though it will be a major-version bump, the release will probably not contain any breaking changes to Guava itself: It will just contain a small breaking change to the much less used guava-testlib, and we keep the version numbers of the two in sync.)

cpovirk avatar May 18 '23 17:05 cpovirk