cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375323642
@xhochy looks like upgrading bison might broken thrift generation on osx:
```
/Users/travis/build/apache/arrow-dist/arrow/parquet-cpp/build/thrift_ep
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375324912
Actually, it's finding the wrong version.
This is an automated message from the Apache Git Service
xhochy commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375325408
I think it also needs a newer `flex` version on OSX, too.
This is an automated message from the Apa
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176440203
##
File path: python-macos/config.sh
##
@@ -21,37 +21,66 @@ set -e
function build_wheel {
echo `pwd`
+export PATH="/
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176440203
##
File path: python-macos/config.sh
##
@@ -21,37 +21,66 @@ set -e
function build_wheel {
echo `pwd`
+export PATH="/
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176441255
##
File path: cpp-linux/debian/rules
##
@@ -23,8 +23,11 @@ override_dh_auto_configure:
-DCMAKE_BUILD_TYPE=$(BUILD_TYPE
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176440203
##
File path: python-macos/config.sh
##
@@ -21,37 +21,66 @@ set -e
function build_wheel {
echo `pwd`
+export PATH="/
kou commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176446024
##
File path: cpp-linux/yum/arrow.spec.in
##
@@ -60,13 +60,17 @@ mkdir cpp/build
cd cpp/build
%cmake3 .. \
-DCMAKE_BUILD_TYPE=$
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176446260
##
File path: python-macos/config.sh
##
@@ -21,37 +21,66 @@ set -e
function build_wheel {
echo `pwd`
+export PATH="/
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176449003
##
File path: cpp-linux/yum/arrow.spec.in
##
@@ -60,13 +60,17 @@ mkdir cpp/build
cd cpp/build
%cmake3 .. \
-DCMAKE_BUILD_TY
xhochy commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176458928
##
File path: python-macos/config.sh
##
@@ -21,37 +21,66 @@ set -e
function build_wheel {
echo `pwd`
+export PATH="/u
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375357003
Why the heck do we keep getting random 404s?
This is an automated message from the Apache Git Serv
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375358552
@xhochy Is there any problem with installing thrift from homebrew?
This is an automated message fr
xhochy commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375360027
@cpcloud We actually want Thrift to use the same namespaced Boost. I would
change our build scripts to explicitly build thrift. Then I'm also able pass in
the BISO
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375360715
> Then I'm also able pass in the BISON and FLEX executable via CMake
variables.
Does that actually work? `find_package(FLEX)` explicitly sets this variable
and
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375360715
> Then I'm also able pass in the BISON and FLEX executable via CMake
variables.
Does that actually work? `find_package(FLEX)` explicitly sets this variable
xhochy commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375377611
> > Then I'm also able pass in the BISON and FLEX executable via CMake
variables.
> Does that actually work? find_package(FLEX) explicitly sets this variable
xhochy commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375382433
`parquet-cpp`'s `thrift_ep` does not pick up the correct Boost version:
https://issues.apache.org/jira/browse/PARQUET-1252 thus I have no moved the
thrift build ou
xhochy commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176509215
##
File path: appveyor.yml
##
@@ -29,11 +29,10 @@ environment:
PYTHON: "3.6"
ARCH: "64"
MSVC_DEFAULT_OPTIONS: ON
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176514837
##
File path: appveyor.yml
##
@@ -29,11 +29,10 @@ environment:
PYTHON: "3.6"
ARCH: "64"
MSVC_DEFAULT_OPTIONS: O
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375405785
@xhochy is there any way we can silence all of the `bcp`/`bjam` output? It
will make for one fewer indirection when debugging travis.
xhochy commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375408993
@cpcloud no idea if they have a be-non-verbose option but I'm ok with piping
their stdout to `/dev/null` as they seem to make their work in a reliable way.
---
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176527524
##
File path: python-macos/config.sh
##
@@ -21,37 +21,94 @@ set -e
function build_wheel {
echo `pwd`
+export PATH="/
xhochy commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176527820
##
File path: python-macos/config.sh
##
@@ -21,37 +21,94 @@ set -e
function build_wheel {
echo `pwd`
+export PATH="/u
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176529648
##
File path: python-macos/config.sh
##
@@ -21,37 +21,94 @@ set -e
function build_wheel {
echo `pwd`
+export PATH="/
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375416775
working on the windows failure now
This is an automated message from the Apache Git Service.
To re
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375430607
Oh man, we are getting so close on the OS X builds!
https://travis-ci.org/apache/arrow-dist/jobs/357034487
Looks like the wheel delocator cannot find
xhochy commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375433610
I think we don't need to run delocate over the wheel, we should bundle all
libs already in the wheel. We could skip that and I can have a try tomorrow to
see if it
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375488961
@xhochy It looks like the wheels include the libs we need:
https://travis-ci.org/cpcloud/arrow-dist/jobs/357120681
Still have to figure out why `setuptools_
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375488961
@xhochy It looks like the wheels include the libs we need:
https://travis-ci.org/cpcloud/arrow-dist/jobs/357120681
Still have to figure out why `setuptools_
cpcloud commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375668074
I think we may somehow have some additional whitespace in the
`Boost_*_LIBRARY` variables on windows. I'm going to try and build a
boost-bundled pyarrow today and
wesm commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176854345
##
File path: scripts/python-wheels-windows.bat
##
@@ -40,7 +37,7 @@ pushd %ARROW_SRC%
@rem fix up symlinks
git config core.symli
wesm commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176854385
##
File path: scripts/python-wheels-windows.bat
##
@@ -54,24 +51,26 @@ mkdir %ARROW_SRC%\cpp\build
pushd %ARROW_SRC%\cpp\build
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176859450
##
File path: scripts/python-wheels-windows.bat
##
@@ -40,7 +37,7 @@ pushd %ARROW_SRC%
@rem fix up symlinks
git config core.sy
wesm commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375830903
You can see why this is broken right here:
https://github.com/apache/arrow/blob/master/python/setup.py#L196
These should not be coupled. I will fix in master a
wesm commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375831209
See https://issues.apache.org/jira/browse/ARROW-2349
This is an automated message from the Apache Git
wesm closed pull request #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork
kou commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375967983
It seems that there are no source version at
https://pypi.python.org/pypi/pyarrow .
Is source upload on PyPI another task?
xhochy commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375968445
We don't have any automation for this yet. We should add a step for this in
the Linux container. If I have time, I can try to add it today.
---
kou commented on issue #23: Build wheels for 0.9.0
URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375968543
I see. Thank!
This is an automated message from the Apache Git Service.
To respond to the message, ple
xhochy opened a new pull request #25: WIP: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub a
xhochy commented on issue #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-377303277
OSX wheels are working again but we probably need to override the version to
0.9.0.post1 to upload it to PyPI.
--
cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25#discussion_r178132589
##
File path: python-macos/config.sh
##
@@ -65,7 +75,23 @@ function build_wheel {
install > /dev/null 2>&1
popd
cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25#discussion_r178132935
##
File path: python-macos/config.sh
##
@@ -153,9 +187,19 @@ function build_wheel {
--bundle-arrow-cpp --bundle-bo
cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25#discussion_r178132846
##
File path: python-macos/config.sh
##
@@ -153,9 +187,19 @@ function build_wheel {
--bundle-arrow-cpp --bundle-bo
cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25#discussion_r178151267
##
File path: python-macos/config.sh
##
@@ -65,7 +75,23 @@ function build_wheel {
install > /dev/null 2>&1
popd
wesm commented on issue #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-377731322
Is this ready to go?
This is an automated message from the Apache Git Service.
To respond to the m
xhochy commented on issue #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-33355
I still need to incorporate @cpcloud comments and we also will need to set
`SETUPTOOLS_SCM_PRETEND_VERSION=0.9.0.post1` as we cannot reupload `0.9.0`
again to Py
xhochy commented on issue #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-378660695
This is now ready to be merged and uploaded to PyPI.
This is an automated message from the Apache
xhochy commented on issue #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-378660872
I will do the upload and verify beforehand that the wheels really work.
This is an automated mess
xhochy closed pull request #25: Fix OSX RPATHs for Boost
URL: https://github.com/apache/arrow-dist/pull/25
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a
kou opened a new pull request #26: [RPM] Remove duplicated -DARROW_ORC=ON
URL: https://github.com/apache/arrow-dist/pull/26
This is an automated message from the Apache Git Service.
To respond to the message, please log on Gi
xhochy closed pull request #26: [RPM] Remove duplicated -DARROW_ORC=ON
URL: https://github.com/apache/arrow-dist/pull/26
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull re
kou opened a new pull request #27: [deb] Stop to use -Wdate-time
URL: https://github.com/apache/arrow-dist/pull/27
Because FlatBuffers uses `__DATE__` and `__TIME__` and it also uses
`-Werror` flag.
See also "timeless" value description of "reproducible" in
http://man7.org/linux/ma
kou commented on issue #27: [deb] Stop to use -Wdate-time
URL: https://github.com/apache/arrow-dist/pull/27#issuecomment-398978251
The build failure isn't related.
This is an automated message from the Apache Git Service.
To r
kou closed pull request #27: [deb] Stop to use -Wdate-time
URL: https://github.com/apache/arrow-dist/pull/27
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from
kou opened a new pull request #28: Support specifying downloaded file name
URL: https://github.com/apache/arrow-dist/pull/28
This is an automated message from the Apache Git Service.
To respond to the message, please log on G
kou commented on issue #28: Support specifying downloaded file name
URL: https://github.com/apache/arrow-dist/pull/28#issuecomment-400542712
The build failure of manylinux1 is unrelated.
This is an automated message from the A
kou closed pull request #28: Support specifying downloaded file name
URL: https://github.com/apache/arrow-dist/pull/28
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull requ
yahonda opened a new pull request #29: ARROW-2768: Support Ubuntu 18.04
URL: https://github.com/apache/arrow-dist/pull/29
This pull request provides Ubuntu 18.04 support.
This is an automated message from the Apache Git Servic
yahonda commented on issue #29: ARROW-2768: Support Ubuntu 18.04
URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401519060
cc @kou who suggested to open a pull request and a JIRA ticket.
This is an automated mes
kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04
URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401662209
Umm. Parallel build may cause build failure.
This is an automated message from the Apache Gi
kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04
URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401663340
I've disabled parallel build. If it works well, I'll merge this.
This is an automated messag
kou closed pull request #29: ARROW-2768: Support Ubuntu 18.04
URL: https://github.com/apache/arrow-dist/pull/29
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (fr
kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04
URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401673189
It works well. I've merged.
This is an automated message from the Apache Git Service.
To res
yahonda commented on issue #29: ARROW-2768: Support Ubuntu 18.04
URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401674535
Thanks for the follow-up and merging.
This is an automated message from the Apache Git S
kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04
URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401697847
I've uploaded packages for Ubuntu 18.04 to packages.red-data-tools.org.
Keep up the good work!
---
kou opened a new pull request #21: Fix .typelib path for Ubuntu trusty
URL: https://github.com/apache/arrow-dist/pull/21
Reported by Mike Sam. Thanks!!!
This is an automated message from the Apache Git Service.
To respond to t
kou commented on issue #21: Fix .typelib path for Ubuntu trusty
URL: https://github.com/apache/arrow-dist/pull/21#issuecomment-360459875
+1
This is an automated message from the Apache Git Service.
To respond to the message, p
kou closed pull request #21: Fix .typelib path for Ubuntu trusty
URL: https://github.com/apache/arrow-dist/pull/21
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request
kou opened a new pull request #22: Drop Ubuntu 17.04 support
URL: https://github.com/apache/arrow-dist/pull/22
Because it's EOL-ed.
This is an automated message from the Apache Git Service.
To respond to the message, please lo
kou closed pull request #22: Drop Ubuntu 17.04 support
URL: https://github.com/apache/arrow-dist/pull/22
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fo
kou commented on issue #22: Drop Ubuntu 17.04 support
URL: https://github.com/apache/arrow-dist/pull/22#issuecomment-360671739
Travis CI is green again. Merged.
This is an automated message from the Apache Git Service.
To resp
yahonda opened a new pull request #30: ARROW-3600: Support Ubuntu 18.10
URL: https://github.com/apache/arrow-dist/pull/30
This pull request provides Ubuntu 18.10 support.
This is an automated message from the Apache Git Servic
kou commented on issue #30: ARROW-3600: Support Ubuntu 18.10
URL: https://github.com/apache/arrow-dist/pull/30#issuecomment-432491625
Thanks! But we've moved build scripts to
https://github.com/apache/arrow/tree/master/dev/tasks/linux-packages .
Can you work on apache/arrow?
---
kou closed pull request #30: ARROW-3600: Support Ubuntu 18.10
URL: https://github.com/apache/arrow-dist/pull/30
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (fr
yahonda commented on issue #30: ARROW-3600: Support Ubuntu 18.10
URL: https://github.com/apache/arrow-dist/pull/30#issuecomment-432492475
Sure. Thanks for the information.
This is an automated message from the Apache Git Serv
andygrove opened a new pull request #1: ARROW-: [Testing] Add test files
that originated in DataFusion
URL: https://github.com/apache/arrow-testing/pull/1
This is an automated message from the Apache Git Service.
To resp
xhochy commented on a change in pull request #1: ARROW-: [Testing] Add test
files that originated in DataFusion
URL: https://github.com/apache/arrow-testing/pull/1#discussion_r252946771
##
File path: data/csv/uk_cities.csv
##
@@ -0,0 +1,37 @@
+"Elgin, Scotland, the UK"
andygrove commented on a change in pull request #1: ARROW-: [Testing] Add
test files that originated in DataFusion
URL: https://github.com/apache/arrow-testing/pull/1#discussion_r253068821
##
File path: data/csv/uk_cities.csv
##
@@ -0,0 +1,37 @@
+"Elgin, Scotland, the
xhochy commented on a change in pull request #1: ARROW-: [Testing] Add test
files that originated in DataFusion
URL: https://github.com/apache/arrow-testing/pull/1#discussion_r253072622
##
File path: data/csv/uk_cities.csv
##
@@ -0,0 +1,37 @@
+"Elgin, Scotland, the UK"
andygrove commented on a change in pull request #1: ARROW-: [Testing] Add
test files that originated in DataFusion
URL: https://github.com/apache/arrow-testing/pull/1#discussion_r253242388
##
File path: data/csv/uk_cities.csv
##
@@ -0,0 +1,37 @@
+"Elgin, Scotland, the
andygrove commented on issue #1: ARROW-: [Testing] Add CSV test file for
use by DataFusion unit tests and examples
URL: https://github.com/apache/arrow-testing/pull/1#issuecomment-459979050
@xhochy I deleted all the files and started again. There is now a single
generated file. I have
xhochy commented on a change in pull request #1: ARROW-: [Testing] Add CSV
test file for use by DataFusion unit tests and examples
URL: https://github.com/apache/arrow-testing/pull/1#discussion_r253297931
##
File path: data/csv/uk_cities.csv
##
@@ -0,0 +1,37 @@
+"Elgin
wesm merged pull request #1: ARROW-: [Testing] Add CSV test file for use by
DataFusion unit tests and examples
URL: https://github.com/apache/arrow-testing/pull/1
This is an automated message from the Apache Git Service.
lidavidm commented on PR #18:
URL: https://github.com/apache/arrow-adbc/pull/18#issuecomment-1164433932
Removed the "old" metadata methods in favor of the new one. I'll merge this
for now but we can keep iterating. Thanks Kirill!
--
This is an automated message from the Apache Git Service
lidavidm closed issue #14: Implement table/column reflection
URL: https://github.com/apache/arrow-adbc/issues/14
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe,
lidavidm merged PR #18:
URL: https://github.com/apache/arrow-adbc/pull/18
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@arrow.apach
lidavidm merged PR #21:
URL: https://github.com/apache/arrow-adbc/pull/21
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@arrow.apach
lidavidm closed issue #20: Improve error handling
URL: https://github.com/apache/arrow-adbc/issues/20
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: is
lidavidm commented on PR #24:
URL: https://github.com/apache/arrow-adbc/pull/24#issuecomment-1164798642
This does raise the question of how Flight SQL deals with it. It's mostly
unspecified (and hence would be autocommit-by-default in practice, presumably).
The Flight SQL driver would repor
lidavidm opened a new pull request, #26:
URL: https://github.com/apache/arrow-adbc/pull/26
Doesn't enable clang-tidy since that needs some integration with the build
process.
Fixes #22.
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
lidavidm commented on issue #23:
URL: https://github.com/apache/arrow-adbc/issues/23#issuecomment-1165550741
We also need to implement rollback
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
lidavidm merged PR #24:
URL: https://github.com/apache/arrow-adbc/pull/24
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@arrow.apach
lidavidm closed issue #23: Clarify transaction semantics
URL: https://github.com/apache/arrow-adbc/issues/23
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-m
lidavidm closed issue #22: Set up linters, sanitizers
URL: https://github.com/apache/arrow-adbc/issues/22
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail
lidavidm merged PR #26:
URL: https://github.com/apache/arrow-adbc/pull/26
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@arrow.apach
lidavidm merged PR #17:
URL: https://github.com/apache/arrow-adbc/pull/17
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@arrow.apach
lidavidm opened a new pull request, #27:
URL: https://github.com/apache/arrow-adbc/pull/27
Make AdbcConnectionNew consistent with AdbcDatabaseNew.
This mostly doesn't affect the driver implementations; it does make the
driver manager a little more complex.
--
This is an automated m
lidavidm commented on PR #27:
URL: https://github.com/apache/arrow-adbc/pull/27#issuecomment-1167432805
CC @hannes. I've left `AdbcStatementNew` alone for now, since that one seems
fully a wash (and it would mean we wouldn't be able to validate the query or
anything until execution time)
201 - 300 of 1554 matches
Mail list logo
