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

MINOR: [Website] Reword ADBC announcement #299

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jduo
Copy link
Member

@jduo jduo commented Jan 17, 2023

Reword the ADBC announcement such that Flight SQL is more clearly specified as being database-agnostic rather than vendor-specific.

Reword the ADBC announcement such that Flight SQL is more clearly specified
as being database-agnostic rather than vendor-specific.
@github-actions
Copy link

Thanks for opening a pull request!

Could you open an issue for this pull request on JIRA?
https://issues.apache.org/jira/browse/ARROW

Then could you also rename pull request title in the following format?

ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}

See also:

Copy link
Member

@lidavidm lidavidm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a notice to the top that the post was edited since its initial publication?

But client applications that want to support multiple database vendors would need to integrate with each of them.
- *Directly use database protocols*.
For some databases, applications can use a database protocol or SDK to directly get Arrow data.
For example, applications could use be written with [Arrow Flight SQL][flight-sql] to connect to Dremio and other databases that support the Flight SQL protocol.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For example, applications could use be written with [Arrow Flight SQL][flight-sql] to connect to Dremio and other databases that support the Flight SQL protocol.
For example, applications could use [Arrow Flight SQL][flight-sql] to connect to Dremio and other databases that support the Flight SQL protocol.

(If you want, I think it's fair to link "Dremio" to the website as well.)

@@ -144,7 +144,7 @@ ADBC fills a specific niche that related projects do not address. It is both:
<tr>
<th></th>
<th class="align-top" style="width: 40%" scope="col">Vendor-neutral (database APIs)</th>
<th class="align-top" style="width: 40%" scope="col">Vendor-specific (database protocols)</th>
<th class="align-top" style="width: 40%" scope="col">Database protocols</th>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's still fair to call them vendor-specific; after all, multiple databases also use the PostgreSQL protocol (it just doesn't have a generic name). Maybe "varies by vendor (database protocols)"?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a better phrase would be "Database specific protocols"

- *Directly use database protocols*.
For some databases, applications can use a database protocol or SDK to directly get Arrow data.
For example, applications could use be written with [Arrow Flight SQL][flight-sql] to connect to Dremio and other databases that support the Flight SQL protocol.
But not all databases support the Flight SQL protocol. An example is Google BigQuery, which has a separate SDK that returns Arrow data. In this case, client applications that want to support additional protocols would need to integrate with each of them.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
But not all databases support the Flight SQL protocol. An example is Google BigQuery, which has a separate SDK that returns Arrow data. In this case, client applications that want to support additional protocols would need to integrate with each of them.
But not all databases support Flight SQL, even if they support Arrow data. An example is Google BigQuery, which has a separate SDK that returns Arrow data. In this case, client applications that want to support additional databases would need to integrate with each of their protocols.

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thank you @jduo

Note there are still some references to "vendor specific / agnostic" in the text such as

- **Vendor-agnostic**: ADBC drivers can implement the API using any underlying protocol, while Flight SQL requires server-side support that may not be easy to add.

If we are going to modify that language, perhaps it would be good to consistently use an alternate term (like "database-agnostic") in all places.

@@ -144,7 +144,7 @@ ADBC fills a specific niche that related projects do not address. It is both:
<tr>
<th></th>
<th class="align-top" style="width: 40%" scope="col">Vendor-neutral (database APIs)</th>
<th class="align-top" style="width: 40%" scope="col">Vendor-specific (database protocols)</th>
<th class="align-top" style="width: 40%" scope="col">Database protocols</th>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a better phrase would be "Database specific protocols"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants