mapleFU commented on code in PR #41362:
URL: https://github.com/apache/arrow/pull/41362#discussion_r157758
##
cpp/src/parquet/column_reader.cc:
##
@@ -1677,40 +1677,43 @@ class TypedRecordReader : public
TypedColumnReaderImpl,
int64_t DelimitRecords(int64_t num_records,
mapleFU commented on PR #41362:
URL: https://github.com/apache/arrow/pull/41362#issuecomment-2074158442
@pitrou @emkornfield @fatemehp @wgtmac
--
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
mapleFU commented on PR #41362:
URL: https://github.com/apache/arrow/pull/41362#issuecomment-2074167343
On My M1Pro MacOS with Release(-O2):
After:
```
Curricane commented on issue #5669:
URL: https://github.com/apache/arrow-rs/issues/5669#issuecomment-2074104509
Is there any progress on this matter?
--
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
tustvold commented on issue #5654:
URL: https://github.com/apache/arrow-rs/issues/5654#issuecomment-2074140467
One way to avoid this leading to subtle downstream breakage might be to do
something along the lines of https://github.com/apache/arrow-rs/issues/3125
--
This is an automated
abandy commented on code in PR #41342:
URL: https://github.com/apache/arrow/pull/41342#discussion_r1577673357
##
swift/Arrow/Package.swift:
##
@@ -36,18 +36,27 @@ let package = Package(
// and therefore doesn't include the unaligned buffer swift changes.
//
aljazerzen commented on issue #5368:
URL: https://github.com/apache/arrow-rs/issues/5368#issuecomment-2074666857
Unfortunately, I agree that because of the bump of `pyo3` (and maybe
`object_store`, didn't check), `arrow` needs a major version bump as well.
This will create a lot of
ZhangHuiGui commented on code in PR #41363:
URL: https://github.com/apache/arrow/pull/41363#discussion_r1577895599
##
cpp/src/arrow/compute/kernels/scalar_if_else.cc:
##
@@ -1195,6 +1195,43 @@ struct ResolveIfElseExec {
}
};
+template
+Result
mcrumiller commented on issue #1755:
URL: https://github.com/apache/arrow-adbc/issues/1755#issuecomment-2075049611
9s for the first, 261ms for the second.
--
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
mapleFU commented on PR #41320:
URL: https://github.com/apache/arrow/pull/41320#issuecomment-2075086943
Thanks! @rouault Would you mind edit this? Or let me handle this with a new
patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
rouault commented on PR #41320:
URL: https://github.com/apache/arrow/pull/41320#issuecomment-2075149363
> Would you mind edit this?
not sure in which direction: your above proposal " // Check all columns has
same row-size" or the alternative proposal I made in
RinChanNOWWW opened a new pull request, #5686:
URL: https://github.com/apache/arrow-rs/pull/5686
# Which issue does this PR close?
Part of #5508.
# Rationale for this change
# What changes are included in this PR?
Support casting from byte
vibhatha commented on PR #41309:
URL: https://github.com/apache/arrow/pull/41309#issuecomment-2074749580
@laurentgo I looked into the debug logs, I am not quite sure what is wrong
here. But would it be possible to check this per module? I know this change is
not that big, but just a
Kimahriman commented on issue #5638:
URL: https://github.com/apache/arrow-rs/issues/5638#issuecomment-2074811609
> > I'm not sure whats @alamb @tustvold opinion, would it make sense to have
your repo in datafusion-contrib @Kimahriman ?
>
> If so, I would be happy to create a repo in
github-actions[bot] commented on PR #41212:
URL: https://github.com/apache/arrow/pull/41212#issuecomment-2074834762
Revision: bed59960750b32f119d3627839473f6ff5be6bda
Submitted crossbow builds: [ursacomputing/crossbow @
vibhatha commented on PR #41210:
URL: https://github.com/apache/arrow/pull/41210#issuecomment-2074833495
@dependabot rebase
--
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.
dependabot[bot] commented on PR #41210:
URL: https://github.com/apache/arrow/pull/41210#issuecomment-2074833600
Sorry, only users with push access can use that command.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
vibhatha commented on PR #41210:
URL: https://github.com/apache/arrow/pull/41210#issuecomment-2074835050
@kou we need to rebase this PR, since there were some fixes for CI failures.
And then we should run the Java crossbows.
Could you please help?
--
This is an automated message from
vibhatha commented on PR #41211:
URL: https://github.com/apache/arrow/pull/41211#issuecomment-2074832059
@kou could we re-run the `java-jars`, not sure if the failure is related or
flaky.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
lidavidm commented on PR #41212:
URL: https://github.com/apache/arrow/pull/41212#issuecomment-2074925949
@dependabot rebase
--
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.
lidavidm commented on PR #41211:
URL: https://github.com/apache/arrow/pull/41211#issuecomment-2074926527
Retried
--
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
mapleFU commented on issue #41317:
URL: https://github.com/apache/arrow/issues/41317#issuecomment-2074947638
I found the root-cause is:
```
/Users/mwish/workspace/CMakeLibs/arrow/cpp/src/arrow/table.cc:622: Check
failed: _s.ok() Operation failed: table_.ValidateFull()
Bad
nealrichardson commented on code in PR #41223:
URL: https://github.com/apache/arrow/pull/41223#discussion_r1577888964
##
r/NEWS.md:
##
@@ -19,6 +19,9 @@
# arrow 16.0.0.9000
+* R functions that users write that use functions that Arrow supports in
dataset queries now can
laurentgo commented on PR #41309:
URL: https://github.com/apache/arrow/pull/41309#issuecomment-2074948712
> It appears Maven is now building things that aren't readable by us for
some reason?
Which is weird but things are also run inside a docker container with volume
mount and the
mcrumiller commented on issue #1755:
URL: https://github.com/apache/arrow-adbc/issues/1755#issuecomment-2074991467
Hi @lidavidm, thanks for your help. I'm new to ADBC so apologies that I'm
not very well-versed here. I'm in a corporate environment so unfortunately I
don't have any admin
felipecrv commented on PR #41320:
URL: https://github.com/apache/arrow/pull/41320#issuecomment-2075058403
> Also cc @felipecrv , do you think TableReader would check the input is
valid? I think the generate side should checks it, and the consumer would
better dcheck that?
Unless the
vibhatha commented on PR #40340:
URL: https://github.com/apache/arrow/pull/40340#issuecomment-2074729269
@github-actions crossbow submit *java*
--
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
leoyvens commented on PR #5683:
URL: https://github.com/apache/arrow-rs/pull/5683#issuecomment-2074961934
Yes, in tonic, the `transport` feature is required for the "batteries
included" server. But arrow-rs does not currently expose a high-level server
API that depends on the tonic
ZhangHuiGui commented on issue #41094:
URL: https://github.com/apache/arrow/issues/41094#issuecomment-2074961032
> If you're interested in adding support for them, that would be greatly
appreciated
Thanks for your suggestion, we'll take a look.
--
This is an automated message from
tustvold commented on PR #5683:
URL: https://github.com/apache/arrow-rs/pull/5683#issuecomment-2074987273
My understanding of the tonic traits was that they were so low level that
whilst in theory one could plugin an alternative this wasn't really practical.
I'm still confused why this
alexandreyc opened a new pull request, #1756:
URL: https://github.com/apache/arrow-adbc/pull/1756
Third PR for the Rust implementation containing the driver exporter which
allows to automatically create a C-compatible driver from a native Rust driver.
CC @mbrobbel
--
This is an
vibhatha commented on PR #41237:
URL: https://github.com/apache/arrow/pull/41237#issuecomment-2074841212
@lidavidm most CIs are failing but the error message is
```bash
Error: The operation was canceled.
```
Could we re-run?
--
This is an automated message from the
lidavidm commented on PR #41210:
URL: https://github.com/apache/arrow/pull/41210#issuecomment-2074925199
@dependabot rebase
--
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.
lidavidm commented on PR #41309:
URL: https://github.com/apache/arrow/pull/41309#issuecomment-2074922927
I mean the error is right there.
```
##[debug][Error: EACCES: permission denied, open
'/home/runner/work/arrow/arrow/java/adapter/avro/target/arrow-avro-17.0.0-SNAPSHOT.jar']
mcrumiller commented on issue #1755:
URL: https://github.com/apache/arrow-adbc/issues/1755#issuecomment-2075082956
First query: 6,667,495 rows in 97.72s.
Second query: 141,851 rows in 11.59s.
--
This is an automated message from the Apache Git Service.
To respond to the message,
felipecrv commented on PR #41320:
URL: https://github.com/apache/arrow/pull/41320#issuecomment-2075081456
> Just curious should we add DCHECK in TableReader helps debugging here
Add a `DCHECK` in `TableBatchReader`, document the precondition (with `///
\pre ...` in the constructor,
zeroshade commented on PR #41180:
URL: https://github.com/apache/arrow/pull/41180#issuecomment-2075165132
@github-actions crossbow submit preview-docs
--
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
tustvold merged PR #5666:
URL: https://github.com/apache/arrow-rs/pull/5666
--
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:
tustvold closed issue #5669: Arrow FLight SQL: invalid location in
get_flight_info_prepared_statement
URL: https://github.com/apache/arrow-rs/issues/5669
--
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
tustvold closed issue #5665: Arrow Flight SQL example server: do_handshake
should include auth header
URL: https://github.com/apache/arrow-rs/issues/5665
--
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
CurtHagenlocher commented on code in PR #1752:
URL: https://github.com/apache/arrow-adbc/pull/1752#discussion_r156163
##
csharp/src/Apache.Arrow.Adbc/C/CAdbcDriverExporter.cs:
##
@@ -708,13 +754,8 @@ public void Dispose()
connection = null;
}
lidavidm merged PR #1754:
URL: https://github.com/apache/arrow-adbc/pull/1754
--
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:
mapleFU commented on PR #41320:
URL: https://github.com/apache/arrow/pull/41320#issuecomment-2075064489
> Is the Parquet reader producing an invalid Table instance and passing it
to TableReader?
Yes. User is running fuzzing on parquet file, when parsing a corrupt parquet
file, we
llama90 commented on issue #41316:
URL: https://github.com/apache/arrow/issues/41316#issuecomment-2074628780
take
--
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
abandy commented on code in PR #41342:
URL: https://github.com/apache/arrow/pull/41342#discussion_r1577679471
##
swift/Arrow/Sources/Arrow/ArrowCExporter.swift:
##
@@ -0,0 +1,134 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license
mapleFU commented on PR #41320:
URL: https://github.com/apache/arrow/pull/41320#issuecomment-2075003778
Also cc @felipecrv , do you think TableReader would check the input is
valid? I think the generate side should checks it, and the consumer would
better dcheck that?
--
This is an
bkietz commented on issue #40342:
URL: https://github.com/apache/arrow/issues/40342#issuecomment-2075084308
Issue resolved by pull request 40356
https://github.com/apache/arrow/pull/40356
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
bkietz merged PR #40356:
URL: https://github.com/apache/arrow/pull/40356
--
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:
vibhatha commented on PR #40340:
URL: https://github.com/apache/arrow/pull/40340#issuecomment-2074766904
@lidavidm I addressed the reviews. And also I added a few additional test
cases for views. And earlier there were 4 tests basically covering the
view-based logics in storing data.
vibhatha commented on PR #40340:
URL: https://github.com/apache/arrow/pull/40340#issuecomment-2074806399
`Dev / Source Release and Merge Script on macos-latest ` CI is keep failing.
I think there is an issue in installing `Ruby`
cc @kou
--
This is an automated message from the Apache
mapleFU commented on PR #41320:
URL: https://github.com/apache/arrow/pull/41320#issuecomment-2074951956
See https://github.com/apache/arrow/issues/41317#issuecomment-2074947638
The root cause is that we didn't check the length. This can be detect by add
the
mapleFU commented on code in PR #40957:
URL: https://github.com/apache/arrow/pull/40957#discussion_r1577957925
##
cpp/src/parquet/encoding.cc:
##
@@ -3988,4 +3988,56 @@ std::unique_ptr MakeDictDecoder(Type::type
type_num,
}
} // namespace detail
+ //
+ //
+bool
mbrobbel commented on PR #1756:
URL: https://github.com/apache/arrow-adbc/pull/1756#issuecomment-2075156735
> I'm not sure why the CI is broken... Maybe a cache invalidation issue?
>
> CC @mbrobbel
Looks like https://github.com/actions/runner-images/issues/9732.
--
This is
Silemo opened a new issue, #5685:
URL: https://github.com/apache/arrow-rs/issues/5685
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
Hi young dev here!
I am currently busy doing an external implementation (or to
github-actions[bot] commented on PR #40340:
URL: https://github.com/apache/arrow/pull/40340#issuecomment-2074733574
Revision: e7cb2d3d240b9b634897f38ff1f3d12772b7031f
Submitted crossbow builds: [ursacomputing/crossbow @
CurtHagenlocher commented on PR #1752:
URL: https://github.com/apache/arrow-adbc/pull/1752#issuecomment-2074799724
This is for external code calling into a C#-implemented driver through the C
API so it doesn't impact Snowflake.
--
This is an automated message from the Apache Git Service.
lidavidm merged PR #1742:
URL: https://github.com/apache/arrow-adbc/pull/1742
--
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 commented on PR #41237:
URL: https://github.com/apache/arrow/pull/41237#issuecomment-2074933102
We have already re-run. The tests themselves appear to get stuck.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
abandy commented on code in PR #41342:
URL: https://github.com/apache/arrow/pull/41342#discussion_r1577677057
##
swift/Arrow/Sources/Arrow/ArrowCExporter.swift:
##
@@ -0,0 +1,134 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license
vibhatha commented on PR #41212:
URL: https://github.com/apache/arrow/pull/41212#issuecomment-2074830020
@github-actions crossbow submit -g java
--
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
mapleFU commented on issue #41339:
URL: https://github.com/apache/arrow/issues/41339#issuecomment-2074922591
Aha, I use master code and run `ReadInBatches` in
`cpp/examples/arrow/parquet_read_write`'s `ReadInBatches`:
```c++
arrow::Status ReadInBatches(std::string path_to_file) {
thisisnic commented on code in PR #41223:
URL: https://github.com/apache/arrow/pull/41223#discussion_r1577938152
##
r/NEWS.md:
##
@@ -19,6 +19,9 @@
# arrow 16.0.0.9000
+* R functions that users write that use functions that Arrow supports in
dataset queries now can be
randolf-scholz commented on issue #28497:
URL: https://github.com/apache/arrow/issues/28497#issuecomment-2075048359
A modulo kernel would also be nice for `duration` types (which are just
integers under the hood). For instance, in time series a common task is
autodetect the frequency at
alexandreyc commented on PR #1756:
URL: https://github.com/apache/arrow-adbc/pull/1756#issuecomment-2075131699
I'm not sure why the CI is broken... Maybe a cache invalidation issue?
CC @mbrobbel
--
This is an automated message from the Apache Git Service.
To respond to the message,
github-actions[bot] commented on PR #41180:
URL: https://github.com/apache/arrow/pull/41180#issuecomment-2075170624
Revision: 32c530e692d239fb659058dc0eacf44c225d65f5
Submitted crossbow builds: [ursacomputing/crossbow @
alamb opened a new issue, #5688:
URL: https://github.com/apache/arrow-rs/issues/5688
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
50.1.0 https://github.com/apache/arrow-rs/issues/5453 was released about 2
months ago:
zanmato1984 commented on code in PR #41335:
URL: https://github.com/apache/arrow/pull/41335#discussion_r1578119685
##
cpp/src/arrow/acero/query_context.cc:
##
@@ -23,6 +23,36 @@ namespace arrow {
using arrow::internal::CpuInfo;
namespace acero {
+namespace internal {
+
mapleFU commented on PR #41366:
URL: https://github.com/apache/arrow/pull/41366#issuecomment-2075282535
Will merge in 2days if no negative comments
--
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
zanmato1984 commented on code in PR #41335:
URL: https://github.com/apache/arrow/pull/41335#discussion_r1578128252
##
cpp/src/arrow/acero/query_context.cc:
##
@@ -23,6 +23,36 @@ namespace arrow {
using arrow::internal::CpuInfo;
namespace acero {
+namespace internal {
+
zeroshade commented on PR #41180:
URL: https://github.com/apache/arrow/pull/41180#issuecomment-2075414280
@github-actions crossbow submit preview-docs
--
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
zeroshade commented on PR #41180:
URL: https://github.com/apache/arrow/pull/41180#issuecomment-2075523237
@github-actions crossbow submit preview-docs
--
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
zanmato1984 commented on code in PR #41335:
URL: https://github.com/apache/arrow/pull/41335#discussion_r1578123814
##
cpp/src/arrow/acero/query_context.cc:
##
@@ -23,6 +23,36 @@ namespace arrow {
using arrow::internal::CpuInfo;
namespace acero {
+namespace internal {
+
raulcd commented on issue #40342:
URL: https://github.com/apache/arrow/issues/40342#issuecomment-2075346442
Shouldn't this be 17.0.0? For some of our downstream packages we might
require some changes (like conda)
--
This is an automated message from the Apache Git Service.
To respond to
zanmato1984 commented on issue #41358:
URL: https://github.com/apache/arrow/issues/41358#issuecomment-2075509417
Acero does support this kind of join, which is enabled by `JoinKeyCmp::IS`:
zanmato1984 commented on issue #41358:
URL: https://github.com/apache/arrow/issues/41358#issuecomment-2075508642
Acero does support this kind of join, which is enabled by `JoinKeyCmp::IS`:
Kimahriman commented on issue #5638:
URL: https://github.com/apache/arrow-rs/issues/5638#issuecomment-2075677843
I'll probably just keep the hdfs-native-object-store crate, so maybe just
use that same name for the repo? don't think `datafusion` really needs to be in
the project name since
Aariq commented on issue #32558:
URL: https://github.com/apache/arrow/issues/32558#issuecomment-2075713199
I suspect this is maybe still an issue? I just had a learner in a workshop
encounter this error on Windows:
```
> library(arrow)
Error: package or namespace load failed
github-actions[bot] commented on PR #41180:
URL: https://github.com/apache/arrow/pull/41180#issuecomment-2075749184
Revision: c106aae233ae504dc5bcf7124a0d418568086fb7
Submitted crossbow builds: [ursacomputing/crossbow @
zeroshade commented on PR #41180:
URL: https://github.com/apache/arrow/pull/41180#issuecomment-2075745836
@github-actions crossbow submit preview-docs
--
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
mapleFU commented on PR #41320:
URL: https://github.com/apache/arrow/pull/41320#issuecomment-2075193327
> not sure in which direction: your above proposal " // Check all columns
has same row-size" or the alternative proposal I made in
rouault commented on PR #41320:
URL: https://github.com/apache/arrow/pull/41320#issuecomment-2075270692
> I mean we can check it here: [#41320
(review)](https://github.com/apache/arrow/pull/41320#pullrequestreview-2019926584)
.
ok, closing that PR, and opening
ianmcook commented on issue #38837:
URL: https://github.com/apache/arrow/issues/38837#issuecomment-2075292973
> I'd be curious what others think of this approach as opposed to actually
making a format change to include statistics alongside the record batches in
the API
I think the
alamb commented on issue #5638:
URL: https://github.com/apache/arrow-rs/issues/5638#issuecomment-2075755439
Created https://github.com/datafusion-contrib/hdfs-native-object-store and
invited you as an admit
--
This is an automated message from the Apache Git Service.
To respond to the
rouault commented on PR #41366:
URL: https://github.com/apache/arrow/pull/41366#issuecomment-2075828859
> is expected to be valid prior to using it with the batch reader
done
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
zeroshade commented on issue #38837:
URL: https://github.com/apache/arrow/issues/38837#issuecomment-2075167964
I'd be curious what others think of this approach as opposed to actually
making a format change to include statistics alongside the record batches in
the API. Particular in the
davidhcoe commented on PR #1730:
URL: https://github.com/apache/arrow-adbc/pull/1730#issuecomment-2075217242
This is ready now, although I am not sure why the `Packaging / Python amd64
macOS (pull_request) ` check is failing. It doesn't _appear_ related to my
changes.
--
This is an
rouault opened a new pull request, #41366:
URL: https://github.com/apache/arrow/pull/41366
### Rationale for this change
Fixes the crash detailed in #41317 in TableBatchReader::ReadNext() on a
corrupted Parquet file
### What changes are included in this PR?
Add a
rouault closed pull request #41320: GH-41317: [C++] Fix crash on invalid
Parquet file
URL: https://github.com/apache/arrow/pull/41320
--
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
github-actions[bot] commented on PR #41359:
URL: https://github.com/apache/arrow/pull/41359#issuecomment-2075364286
:warning: GitHub issue #41367 **has been automatically assigned in GitHub**
to PR creator.
--
This is an automated message from the Apache Git Service.
To respond to the
ClifHouck commented on code in PR #40957:
URL: https://github.com/apache/arrow/pull/40957#discussion_r1578252008
##
cpp/src/parquet/encoding.cc:
##
@@ -3988,4 +3988,56 @@ std::unique_ptr MakeDictDecoder(Type::type
type_num,
}
} // namespace detail
+ //
+ //
+bool
anjakefala commented on PR #41041:
URL: https://github.com/apache/arrow/pull/41041#issuecomment-2075463670
`setuptools_scm` does have logging. I'm going to see if the logging helps
reveal anything. If nothing, I'll explore the alternatives!
--
This is an automated message from the Apache
CurtHagenlocher commented on issue #1490:
URL: https://github.com/apache/arrow-adbc/issues/1490#issuecomment-207540
It would be good to iron out the exact scope of what a 1.0.0 would include.
I'd like (for instance) to make some breaking changes to some C# APIs to clean
them up a bit
zanmato1984 commented on code in PR #41335:
URL: https://github.com/apache/arrow/pull/41335#discussion_r1578128252
##
cpp/src/arrow/acero/query_context.cc:
##
@@ -23,6 +23,36 @@ namespace arrow {
using arrow::internal::CpuInfo;
namespace acero {
+namespace internal {
+
jbonofre commented on PR #41187:
URL: https://github.com/apache/arrow/pull/41187#issuecomment-2075391064
Sure thing ! I'm traveling this week but I should find time ;)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
alamb commented on issue #5638:
URL: https://github.com/apache/arrow-rs/issues/5638#issuecomment-2075535831
https://github.com/datafusion-contrib/datafusion-objectstore-hdfs already
exists
Would you like to make one like
zeroshade commented on code in PR #1722:
URL: https://github.com/apache/arrow-adbc/pull/1722#discussion_r1578500656
##
go/adbc/driver/bigquery/connection.go:
##
@@ -0,0 +1,868 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license
kou commented on PR #41359:
URL: https://github.com/apache/arrow/pull/41359#issuecomment-2075824614
The "AMD64 macOS 12 GLib & Ruby" failure: #41369
--
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
leoyvens commented on PR #5683:
URL: https://github.com/apache/arrow-rs/pull/5683#issuecomment-2075300691
For arrow-flight it is truly only required for the high-level client, not
for the server since `FlightServiceServer` is just a `Service` trait
implementation. But I'm happy to rename
mapleFU commented on PR #41187:
URL: https://github.com/apache/arrow/pull/41187#issuecomment-2075397876
Oh, sorry for bothering, enjoy your traveling!
--
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
github-actions[bot] commented on PR #41180:
URL: https://github.com/apache/arrow/pull/41180#issuecomment-2075417969
Revision: f5f644c596a599f318fcc9060d51e7cdcfc1d1ca
Submitted crossbow builds: [ursacomputing/crossbow @
1 - 100 of 214 matches
Mail list logo