+1
On Sat, Apr 6, 2024, at 22:20, Matt Topol wrote: > +1 > > On Sat, Apr 6, 2024, 4:54 AM Andrew Lamb <[email protected]> wrote: > >> +1 >> >> On Fri, Apr 5, 2024 at 9:55 PM Jacob Wujciak <[email protected]> >> wrote: >> >> > + 1 (non-binding) >> > >> > Am Sa., 6. Apr. 2024 um 01:57 Uhr schrieb Joel Lubinitsky < >> > [email protected]>: >> > >> > > Yes, just updated both the issue and the PR. >> > > >> > > Thanks, >> > > Joel >> > > >> > > On Fri, Apr 5, 2024 at 7:51 PM Sutou Kouhei <[email protected]> >> wrote: >> > > >> > > > +1 >> > > > >> > > > Could you also update the description of >> > > > https://github.com/apache/arrow-adbc/issues/1650 ? >> > > > >> > > > Thanks, >> > > > -- >> > > > kou >> > > > >> > > > In <[email protected]> >> > > > "Re: [VOTE] Add new info codes and options keys to ADBC >> > specification" >> > > > on Fri, 05 Apr 2024 15:39:33 -0000, >> > > > Joel Lubinitsky <[email protected]> wrote: >> > > > >> > > > > Update on this: >> > > > > >> > > > > I've removed ADBC_INFO_VENDOR_READ_ONLY from the proposal. The >> change >> > > is >> > > > reflected in this commit [1] on the original PR [2]. The numbers >> > > > corresponding to each of the other info codes have been decremented >> by >> > 1 >> > > to >> > > > fill the gap in numbering. >> > > > > >> > > > > The reason is that a similar option already exists via >> > > > ConnectionGet/SetOptions, so defining it on the driver isn't helpful. >> > > > > >> > > > > [1]: >> > > > >> > > >> > >> https://github.com/apache/arrow-adbc/pull/1649/commits/a52a4fa16e6b740392d3617751e28f044f1a8325 >> > > > > [2]: https://github.com/apache/arrow-adbc/pull/1649 >> > > > > >> > > > > Thanks, >> > > > > Joel >> > > > > >> > > > > On 2024/04/03 11:01:13 Joel Lubinitsky wrote: >> > > > >> Hello, >> > > > >> >> > > > >> I would like to propose a change to the ADBC specification that >> > > > introduces >> > > > >> 5 new standard info codes and formalizes 3 existing option keys. >> > > > >> >> > > > >> The info codes being introduced are: >> > > > >> - ADBC_INFO_VENDOR_READ_ONLY 3 >> > > > >> - ADBC_INFO_VENDOR_SQL 4 >> > > > >> - ADBC_INFO_VENDOR_SUBSTRAIT 5 >> > > > >> - ADBC_INFO_VENDOR_SUBSTRAIT_MIN_VERSION 6 >> > > > >> - ADBC_INFO_VENDOR_SUBSTRAIT_MAX_VERSION 7 >> > > > >> >> > > > >> The option keys have been in use (defined in options.h) and are >> > being >> > > > moved >> > > > >> to adbc.h: >> > > > >> - ADBC_INGEST_OPTION_TARGET_CATALOG "adbc.ingest.target_catalog" >> > > > >> - ADBC_INGEST_OPTION_TARGET_DB_SCHEMA >> "adbc.ingest.target_db_schema" >> > > > >> - ADBC_INGEST_OPTION_TEMPORARY "adbc.ingest.temporary" >> > > > >> >> > > > >> The change is described in this issue [0] and an implementation is >> > > > included >> > > > >> in this PR [1]. >> > > > >> >> > > > >> The vote will be open for at least 72 hours. >> > > > >> >> > > > >> [ ] +1 Add these info codes and options keys to the ADBC spec >> > > > >> [ ] +0 >> > > > >> [ ] -1 Do not add these to the ADBC spec because... >> > > > >> >> > > > >> Thanks, >> > > > >> Joel >> > > > >> >> > > > >> [0]: https://github.com/apache/arrow-adbc/issues/1650 >> > > > >> [1]: https://github.com/apache/arrow-adbc/pull/1649 >> > > > >> >> > > > >> > > >> > >>
