-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
ARROW-16789: [Format] Remove experimental marker from C Streaming Interface #13345
ARROW-16789: [Format] Remove experimental marker from C Streaming Interface #13345
Conversation
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW Opening JIRAs ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename pull request title in the following format?
or
See also: |
@@ -36,8 +36,7 @@ | |||
/** | |||
* Functions for working with the C data interface. | |||
* <p> | |||
* This API is EXPERIMENTAL. Note that currently only 64bit systems are | |||
* supported. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to remove this here? The fact that the Java API is experimental is a bit unrelated to the status of the C stream interface.
cc @zhztheplayer for opinions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes afaik we don't yet have any codes about C Stream Interface in Java.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay sorry I misinterpreted that. I will revert.
(This also resolves https://issues.apache.org/jira/browse/ARROW-14613) |
@@ -38,6 +38,7 @@ | |||
* <p> | |||
* This API is EXPERIMENTAL. Note that currently only 64bit systems are | |||
* supported. | |||
* Note that currently only 64bit systems are supported. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line was duplicated by mistake? :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @wjones127 !
Approved by the PMC: https://lists.apache.org/thread/39wsd1y8f7fk1k44krm5jrw3pv6dnm45