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

[Java] Remove Contrib/Experimental from Flight SQL #39037

Closed
laurentgo opened this issue Dec 1, 2023 · 4 comments · Fixed by #39040
Closed

[Java] Remove Contrib/Experimental from Flight SQL #39037

laurentgo opened this issue Dec 1, 2023 · 4 comments · Fixed by #39040

Comments

@laurentgo
Copy link
Collaborator

Describe the enhancement requested

Remove the (Contrib/Experimental) mention in Java Flight SQL modules which also show up in Maven Central website, as those modules have been part of the project for a while now.

Component(s)

Java

laurentgo added a commit to laurentgo/arrow that referenced this issue Dec 1, 2023
…t SQL

Considering that Flight SQL has been present for a while and should be
fairly stable, remove the `(Contrib/Experimental)` mention from the pom
file which also shows up on Maven Central UI pages
@jbonofre
Copy link
Member

jbonofre commented Dec 3, 2023

Thanks, I gonna take a look.

@lidavidm lidavidm changed the title Remove Contrib/Experimental [Java] Remove Contrib/Experimental Dec 14, 2023
@lidavidm lidavidm changed the title [Java] Remove Contrib/Experimental [Java] Remove Contrib/Experimental from Flight SQL Dec 14, 2023
lidavidm pushed a commit that referenced this issue Dec 14, 2023
…39040)

### Rationale for this change

Considering that Flight SQL has been present for a while and should be fairly stable, remove the `(Contrib/Experimental)` mention from the pom file which also shows up on Maven Central UI pages

### Are these changes tested?

Local test but there's no code change, only cosmetic

### Are there any user-facing changes?

None

* Closes: #39037

Authored-by: Laurent Goujon <laurent@apache.org>
Signed-off-by: David Li <li.davidm96@gmail.com>
@lidavidm lidavidm added this to the 15.0.0 milestone Dec 14, 2023
@jbonofre
Copy link
Member

I guess documentation should be updated as well. For instance https://arrow.apache.org/docs/format/FlightSql.html still mention experimental.
I can do a pass if you want 😄

@lidavidm
Copy link
Member

I had filed #39204, feel free to take it on. Thanks!

@jbonofre
Copy link
Member

@lidavidm cool ! Thanks ! I'm tackling it 😄

clayburn pushed a commit to clayburn/arrow that referenced this issue Jan 23, 2024
…t SQL (apache#39040)

### Rationale for this change

Considering that Flight SQL has been present for a while and should be fairly stable, remove the `(Contrib/Experimental)` mention from the pom file which also shows up on Maven Central UI pages

### Are these changes tested?

Local test but there's no code change, only cosmetic

### Are there any user-facing changes?

None

* Closes: apache#39037

Authored-by: Laurent Goujon <laurent@apache.org>
Signed-off-by: David Li <li.davidm96@gmail.com>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
…t SQL (apache#39040)

### Rationale for this change

Considering that Flight SQL has been present for a while and should be fairly stable, remove the `(Contrib/Experimental)` mention from the pom file which also shows up on Maven Central UI pages

### Are these changes tested?

Local test but there's no code change, only cosmetic

### Are there any user-facing changes?

None

* Closes: apache#39037

Authored-by: Laurent Goujon <laurent@apache.org>
Signed-off-by: David Li <li.davidm96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants