liyafan82 commented on a change in pull request #6323:
URL: https://github.com/apache/arrow/pull/6323#discussion_r414332379
##
File path:
java/memory/src/test/java/org/apache/arrow/memory/TestLargeArrowBuf.java
##
@@ -0,0 +1,68 @@
+/*
+ * Licensed to the Apache Software Founda
liyafan82 commented on a change in pull request #6323:
URL: https://github.com/apache/arrow/pull/6323#discussion_r414330656
##
File path:
java/memory/src/main/java/org/apache/arrow/memory/NettyAllocationManager.java
##
@@ -34,31 +33,34 @@
static final UnsafeDirectLittleEndi
liyafan82 commented on a change in pull request #6323:
URL: https://github.com/apache/arrow/pull/6323#discussion_r414330471
##
File path:
java/memory/src/main/java/org/apache/arrow/memory/NettyAllocationManager.java
##
@@ -34,31 +33,34 @@
static final UnsafeDirectLittleEndi
liyafan82 commented on a change in pull request #6323:
URL: https://github.com/apache/arrow/pull/6323#discussion_r414330317
##
File path:
java/memory/src/main/java/org/apache/arrow/memory/NettyAllocationManager.java
##
@@ -34,31 +33,34 @@
static final UnsafeDirectLittleEndi
mcassels commented on a change in pull request #6770:
URL: https://github.com/apache/arrow/pull/6770#discussion_r414276861
##
File path: rust/datafusion/src/logicalplan.rs
##
@@ -828,8 +828,8 @@ mod tests {
.build()?;
let expected = "Projection: #id\
-
mcassels commented on a change in pull request #6770:
URL: https://github.com/apache/arrow/pull/6770#discussion_r414276380
##
File path: rust/datafusion/src/utils.rs
##
@@ -120,6 +143,7 @@ pub fn array_value_to_string(column: array::ArrayRef, row:
usize) -> Result {
cyb70289 commented on pull request #6954:
URL: https://github.com/apache/arrow/pull/6954#issuecomment-618782973
> Is the function `Armv8CrcHashParallel` used somewhere? Sorry if I overlook
it.
It's not used. Actually the whole file hash_util.h is not used per [this
comment](https://
paddyhoran commented on a change in pull request #7004:
URL: https://github.com/apache/arrow/pull/7004#discussion_r414241189
##
File path: rust/arrow/src/array/mod.rs
##
@@ -85,6 +85,7 @@ mod array;
mod builder;
mod data;
mod equal;
+mod union;
Review comment:
Yea, e
paddyhoran commented on a change in pull request #7004:
URL: https://github.com/apache/arrow/pull/7004#discussion_r414240892
##
File path: rust/arrow/src/array/equal.rs
##
@@ -1046,6 +1062,30 @@ impl PartialEq for Value {
}
}
+impl JsonEqual for UnionArray {
+fn equ
paddyhoran commented on pull request #7004:
URL: https://github.com/apache/arrow/pull/7004#issuecomment-618761556
@andygrove just going to leave a general comment as it's all related.
Overall, I felt this PR was getting big, I was trying to avoid getting into
the IPC stuff in this PR
zgramana edited a comment on pull request #6121:
URL: https://github.com/apache/arrow/pull/6121#issuecomment-618756547
@eerhardt I'd like to chime in to say that I have just come across this
conversation late--and *after* implementing an alternative approach which much
more in-line with ot
sunchao commented on a change in pull request #6935:
URL: https://github.com/apache/arrow/pull/6935#discussion_r414235616
##
File path: rust/parquet/src/column/reader.rs
##
@@ -190,15 +190,12 @@ impl ColumnReaderImpl {
(self.num_buffered_values - self.num_d
zgramana edited a comment on pull request #6121:
URL: https://github.com/apache/arrow/pull/6121#issuecomment-618756547
@eerhardt I'd like to chime in to say that I have just come across this
conversation late--and *after* implementing an alternative approach which much
more in-line with ot
zgramana edited a comment on pull request #6121:
URL: https://github.com/apache/arrow/pull/6121#issuecomment-618756547
@eerhardt I'd like to chime in to say that I have just come across this
conversation late--and *after* implementing an alternative approach which much
more in-line with ot
zgramana commented on pull request #6121:
URL: https://github.com/apache/arrow/pull/6121#issuecomment-618756547
@eerhardt I'd like to chime in that I have just come across this
conversation after implementing an alternative approach much more in line with
other Arrow language implementatio
paddyhoran commented on a change in pull request #6306:
URL: https://github.com/apache/arrow/pull/6306#discussion_r414234200
##
File path: rust/arrow/src/compute/kernels/sort.rs
##
@@ -0,0 +1,671 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more cont
paddyhoran commented on a change in pull request #6980:
URL: https://github.com/apache/arrow/pull/6980#discussion_r414230710
##
File path: rust/arrow/src/array/builder.rs
##
@@ -236,6 +251,14 @@ impl BufferBuilderTrait for
BufferBuilder {
self.write_bytes(v.to_byte_sl
paddyhoran edited a comment on pull request #7024:
URL: https://github.com/apache/arrow/pull/7024#issuecomment-618749586
CI is failing again, I thought this was fixed by #7010
This is an automated message from the Apache Git
paddyhoran commented on pull request #7024:
URL: https://github.com/apache/arrow/pull/7024#issuecomment-618749586
CI is failing again, I thought this was fixed by #8558
This is an automated message from the Apache Git Servic
sunchao edited a comment on pull request #6949:
URL: https://github.com/apache/arrow/pull/6949#issuecomment-618740530
Yes I think it is beneficial to avoid dropping buffers with `seek`, although
it will be nice if the `seek_relative` will be stabilized soon so we can just
use that.
sunchao commented on pull request #6949:
URL: https://github.com/apache/arrow/pull/6949#issuecomment-618740530
Yes I think it is beneficial to avoid dropping buffers with `seek`, although
it will be nice if the `seek_relative` will be stabilized soon so we can just
use that.
> Also,
github-actions[bot] commented on pull request #7028:
URL: https://github.com/apache/arrow/pull/7028#issuecomment-618731413
https://issues.apache.org/jira/browse/ARROW-8575
This is an automated message from the Apache Git Serv
nealrichardson opened a new pull request #7028:
URL: https://github.com/apache/arrow/pull/7028
Instead of adding a PR comment of "This needs rebase" and wait for the
author to get around to it, with this workflow you can just type "rebase" and
GHA will do it for you. If it rebases cleanly,
bkietz commented on a change in pull request #7026:
URL: https://github.com/apache/arrow/pull/7026#discussion_r414184803
##
File path: r/src/expression.cpp
##
@@ -21,99 +21,97 @@
// [[arrow::export]]
std::shared_ptr dataset___expr__field_ref(std::string name) {
- return st
bkietz commented on a change in pull request #7026:
URL: https://github.com/apache/arrow/pull/7026#discussion_r414184803
##
File path: r/src/expression.cpp
##
@@ -21,99 +21,97 @@
// [[arrow::export]]
std::shared_ptr dataset___expr__field_ref(std::string name) {
- return st
nealrichardson commented on a change in pull request #7026:
URL: https://github.com/apache/arrow/pull/7026#discussion_r414153717
##
File path: r/src/expression.cpp
##
@@ -21,99 +21,97 @@
// [[arrow::export]]
std::shared_ptr dataset___expr__field_ref(std::string name) {
- r
BryanCutler commented on a change in pull request #6323:
URL: https://github.com/apache/arrow/pull/6323#discussion_r414120706
##
File path:
java/memory/src/main/java/org/apache/arrow/memory/NettyAllocationManager.java
##
@@ -34,31 +33,34 @@
static final UnsafeDirectLittleEn
wesm commented on pull request #6992:
URL: https://github.com/apache/arrow/pull/6992#issuecomment-618678293
Sweet thanks, merging now
This is an automated message from the Apache Git Service.
To respond to the message, please
jorisvandenbossche commented on pull request #6992:
URL: https://github.com/apache/arrow/pull/6992#issuecomment-618674669
I further cleaned up the shim to remove if/else checks we no longer need, so
should be ready now.
This
github-actions[bot] commented on pull request #7027:
URL: https://github.com/apache/arrow/pull/7027#issuecomment-618659884
https://issues.apache.org/jira/browse/ARROW-8572
This is an automated message from the Apache Git Serv
lidavidm opened a new pull request #7027:
URL: https://github.com/apache/arrow/pull/7027
- Adds an explicit range check to `UnionArray.child`
- Exposes `child`, `value_offsets`, and `type_codes` to Python. (In Python,
they're wrapped in arrays for you to save you the trouble.)
Thi
github-actions[bot] commented on pull request #7025:
URL: https://github.com/apache/arrow/pull/7025#issuecomment-618651266
https://issues.apache.org/jira/browse/ARROW-2260
This is an automated message from the Apache Git Serv
github-actions[bot] commented on pull request #7026:
URL: https://github.com/apache/arrow/pull/7026#issuecomment-618651261
https://issues.apache.org/jira/browse/ARROW-7391
This is an automated message from the Apache Git Serv
bkietz opened a new pull request #7026:
URL: https://github.com/apache/arrow/pull/7026
Serialization is implemented by converting Expressions to Arrays then
writing a tiny IPC file. This is a ridiculous way to serialize Expressions but
it should be acceptable since these classes are destin
chrish42 opened a new pull request #7025:
URL: https://github.com/apache/arrow/pull/7025
The following patch ads Gflags support to `plasma-store-server`, leaves out
the backtraces on invalid command-line options, and generally tries to make the
error messages more useful in terms of nudgin
github-actions[bot] commented on pull request #7024:
URL: https://github.com/apache/arrow/pull/7024#issuecomment-618624379
https://issues.apache.org/jira/browse/ARROW-8573
This is an automated message from the Apache Git Serv
andygrove opened a new pull request #7024:
URL: https://github.com/apache/arrow/pull/7024
Now that Rust 1.43.0 is released, we should upgrade to 1.44 nightly. It
looks like there were changes in rustfmt rules.
This is an aut
andygrove commented on pull request #7018:
URL: https://github.com/apache/arrow/pull/7018#issuecomment-618605512
@nevi-me This is looking good, but the generated source file needs the ASF
header. CI is failing with ` apache-rat license violation:
rust/arrow-flight/src/arrow.flight.protocol
markhildreth commented on pull request #6972:
URL: https://github.com/apache/arrow/pull/6972#issuecomment-618598341
@andygrove Yup, I was planning on doing that in a separate PR. If you'd like
I can do that in this one.
This
andygrove commented on pull request #6972:
URL: https://github.com/apache/arrow/pull/6972#issuecomment-618597619
@markhildreth This looks great, but is now duplicating the code between
arrow and datafusion. Can we remove the datafusion utils copy and have
datafusion use the arrow utils ins
mayuropensource commented on issue #7020:
URL: https://github.com/apache/arrow/pull/7020#issuecomment-618577963
@wesm sure thing, I'll keep that in mind in the future.
This is an automated message from the Apache Git Service.
mayuropensource commented on a change in pull request #7022:
URL: https://github.com/apache/arrow/pull/7022#discussion_r414031141
##
File path: cpp/src/arrow/io/caching.h
##
@@ -27,6 +27,44 @@
namespace arrow {
namespace io {
+
+struct ARROW_EXPORT CacheOptions {
+ static
houqp commented on a change in pull request #7009:
URL: https://github.com/apache/arrow/pull/7009#discussion_r414007919
##
File path: rust/parquet/src/record/api.rs
##
@@ -50,6 +50,33 @@ impl Row {
pub fn len(&self) -> usize {
self.fields.len()
}
+
+pub f
xhochy commented on issue #6992:
URL: https://github.com/apache/arrow/pull/6992#issuecomment-618538122
👍
2 years ago released `pandas` version still sounds very generous. People who
cannot upgrade from that to a newer version will probably have the same
problems with `pyarrow` upda
lidavidm commented on a change in pull request #7022:
URL: https://github.com/apache/arrow/pull/7022#discussion_r413990293
##
File path: cpp/src/arrow/io/caching.h
##
@@ -27,6 +27,44 @@
namespace arrow {
namespace io {
+
+struct ARROW_EXPORT CacheOptions {
+ static constex
wesm commented on issue #6992:
URL: https://github.com/apache/arrow/pull/6992#issuecomment-618537844
Actually I'll hold off on merging this to confirm that @jorisvandenbossche
has done everything that he planned
This is an a
github-actions[bot] commented on issue #7023:
URL: https://github.com/apache/arrow/pull/7023#issuecomment-618536730
https://issues.apache.org/jira/browse/ARROW-8571
This is an automated message from the Apache Git Service.
To
wesm commented on issue #6992:
URL: https://github.com/apache/arrow/pull/6992#issuecomment-618536764
The Appveyor failure is unrelated
This is an automated message from the Apache Git Service.
To respond to the message, pleas
tustvold commented on a change in pull request #6980:
URL: https://github.com/apache/arrow/pull/6980#discussion_r413987657
##
File path: rust/arrow/src/array/builder.rs
##
@@ -236,6 +251,14 @@ impl BufferBuilderTrait for
BufferBuilder {
self.write_bytes(v.to_byte_slic
pitrou commented on a change in pull request #7023:
URL: https://github.com/apache/arrow/pull/7023#discussion_r413987573
##
File path: appveyor.yml
##
@@ -61,7 +61,7 @@ environment:
- JOB: "Build"
GENERATOR: Ninja
CONFIGURATION: "Release"
- APPVEYOR_BUIL
markhildreth edited a comment on issue #6972:
URL: https://github.com/apache/arrow/pull/6972#issuecomment-618506903
From a purely practical standpoint, this PR is ready for further review and
merging. If approved, I would probably add some minor JIRA issue for the
following:
* Trying to
markhildreth edited a comment on issue #6972:
URL: https://github.com/apache/arrow/pull/6972#issuecomment-618506903
From a purely practical standpoint, this PR is ready for further review and
merging. If approved, I would probably add some minor JIRA issue for the
following:
* Trying to
xhochy opened a new pull request #7023:
URL: https://github.com/apache/arrow/pull/7023
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
github-actions[bot] commented on issue #7019:
URL: https://github.com/apache/arrow/pull/7019#issuecomment-618528458
Revision: 5bcfeab4c9bacc0b3a262a7522bfaf985025d3ec
Submitted crossbow builds: [ursa-labs/crossbow @
actions-165](https://github.com/ursa-labs/crossbow/branches/all?quer
nealrichardson commented on issue #7019:
URL: https://github.com/apache/arrow/pull/7019#issuecomment-618527651
@github-actions crossbow submit homebrew-cpp
This is an automated message from the Apache Git Service.
To respond
markhildreth commented on a change in pull request #7006:
URL: https://github.com/apache/arrow/pull/7006#discussion_r413958397
##
File path: rust/arrow/src/array/array.rs
##
@@ -2592,6 +2619,15 @@ mod tests {
assert_eq!(DataType::Int32, list_array.value_type());
markhildreth commented on a change in pull request #7006:
URL: https://github.com/apache/arrow/pull/7006#discussion_r413958397
##
File path: rust/arrow/src/array/array.rs
##
@@ -2592,6 +2619,15 @@ mod tests {
assert_eq!(DataType::Int32, list_array.value_type());
markhildreth commented on a change in pull request #7006:
URL: https://github.com/apache/arrow/pull/7006#discussion_r413958397
##
File path: rust/arrow/src/array/array.rs
##
@@ -2592,6 +2619,15 @@ mod tests {
assert_eq!(DataType::Int32, list_array.value_type());
vertexclique commented on a change in pull request #6980:
URL: https://github.com/apache/arrow/pull/6980#discussion_r413972358
##
File path: rust/arrow/src/array/builder.rs
##
@@ -236,6 +251,14 @@ impl BufferBuilderTrait for
BufferBuilder {
self.write_bytes(v.to_byte_
kiszk commented on a change in pull request #6985:
URL: https://github.com/apache/arrow/pull/6985#discussion_r413970301
##
File path: cpp/src/parquet/level_conversion.h
##
@@ -0,0 +1,191 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor l
tustvold commented on a change in pull request #6980:
URL: https://github.com/apache/arrow/pull/6980#discussion_r413969298
##
File path: rust/arrow/src/array/builder.rs
##
@@ -236,6 +251,14 @@ impl BufferBuilderTrait for
BufferBuilder {
self.write_bytes(v.to_byte_slic
BryanCutler commented on issue #6992:
URL: https://github.com/apache/arrow/pull/6992#issuecomment-618517973
Sounds good to me. FWIW, Spark also has a minimum Pandas version set at
0.23.2.
This is an automated message from th
kiszk commented on a change in pull request #6985:
URL: https://github.com/apache/arrow/pull/6985#discussion_r413960907
##
File path: cpp/src/parquet/level_conversion.h
##
@@ -0,0 +1,191 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor l
bryantbiggs commented on issue #4140:
URL: https://github.com/apache/arrow/pull/4140#issuecomment-618512883
thanks @andygrove !
This is an automated message from the Apache Git Service.
To respond to the message, please log o
markhildreth commented on a change in pull request #7006:
URL: https://github.com/apache/arrow/pull/7006#discussion_r413958397
##
File path: rust/arrow/src/array/array.rs
##
@@ -2592,6 +2619,15 @@ mod tests {
assert_eq!(DataType::Int32, list_array.value_type());
github-actions[bot] commented on issue #7022:
URL: https://github.com/apache/arrow/pull/7022#issuecomment-618510578
https://issues.apache.org/jira/browse/ARROW-8562
This is an automated message from the Apache Git Service.
To
kiszk commented on a change in pull request #6985:
URL: https://github.com/apache/arrow/pull/6985#discussion_r413954827
##
File path: cpp/src/parquet/level_conversion_test.cc
##
@@ -0,0 +1,162 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contrib
wesm commented on issue #7020:
URL: https://github.com/apache/arrow/pull/7020#issuecomment-618508772
@mayuropensource it's not necessary to open a new PR when you want to redo
your commits, you can just force push your branch
---
markhildreth commented on a change in pull request #6972:
URL: https://github.com/apache/arrow/pull/6972#discussion_r413951619
##
File path: rust/parquet/src/encodings/rle.rs
##
@@ -830,7 +826,7 @@ mod tests {
values.clear();
let mut rng = thread_rng()
kiszk commented on a change in pull request #6985:
URL: https://github.com/apache/arrow/pull/6985#discussion_r413954827
##
File path: cpp/src/parquet/level_conversion_test.cc
##
@@ -0,0 +1,162 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contrib
kiszk commented on a change in pull request #6985:
URL: https://github.com/apache/arrow/pull/6985#discussion_r413953833
##
File path: cpp/src/parquet/column_reader.cc
##
@@ -50,6 +51,140 @@ using arrow::internal::checked_cast;
namespace parquet {
+namespace {
+
+inline voi
markhildreth edited a comment on issue #6972:
URL: https://github.com/apache/arrow/pull/6972#issuecomment-618501806
@andygrove Thanks for the feedback. I have updated the PR with a less leaky
API. I also tweaked the parquet test to workaround the new type inference
changes.
@nevi-me
markhildreth commented on issue #6972:
URL: https://github.com/apache/arrow/pull/6972#issuecomment-618506903
From a purely practical standpoint, this PR is ready for further review and
merging. If approved, I would probably add some minor issue for the following:
* Trying to avoid the ty
vertexclique commented on a change in pull request #6980:
URL: https://github.com/apache/arrow/pull/6980#discussion_r413951534
##
File path: rust/arrow/src/array/builder.rs
##
@@ -236,6 +251,14 @@ impl BufferBuilderTrait for
BufferBuilder {
self.write_bytes(v.to_byte_
vertexclique commented on a change in pull request #6980:
URL: https://github.com/apache/arrow/pull/6980#discussion_r413951534
##
File path: rust/arrow/src/array/builder.rs
##
@@ -236,6 +251,14 @@ impl BufferBuilderTrait for
BufferBuilder {
self.write_bytes(v.to_byte_
markhildreth commented on a change in pull request #6972:
URL: https://github.com/apache/arrow/pull/6972#discussion_r413951619
##
File path: rust/parquet/src/encodings/rle.rs
##
@@ -830,7 +826,7 @@ mod tests {
values.clear();
let mut rng = thread_rng()
vertexclique commented on a change in pull request #6980:
URL: https://github.com/apache/arrow/pull/6980#discussion_r413951534
##
File path: rust/arrow/src/array/builder.rs
##
@@ -236,6 +251,14 @@ impl BufferBuilderTrait for
BufferBuilder {
self.write_bytes(v.to_byte_
markhildreth commented on issue #6972:
URL: https://github.com/apache/arrow/pull/6972#issuecomment-618501806
@andygrove Thanks for the feedback. I have updated the PR with a less leaky
API. I also fixed the type inference problem that was caused by the new
dependency.
@nevi-me True
markhildreth edited a comment on issue #6972:
URL: https://github.com/apache/arrow/pull/6972#issuecomment-618501806
@andygrove Thanks for the feedback. I have updated the PR with a less leaky
API. I also fixed the type inference problem that was caused by the new
dependency.
@nevi-m
mayuropensource opened a new pull request #7022:
URL: https://github.com/apache/arrow/pull/7022
_(Recreating the PR from a clean repo, sorry about earlier PR which was not
cleanly merged)._
**JIRA:** https://issues.apache.org/jira/browse/ARROW-8562
This change is not actually
github-actions[bot] commented on issue #7021:
URL: https://github.com/apache/arrow/pull/7021#issuecomment-618494095
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 al
lidavidm commented on a change in pull request #7020:
URL: https://github.com/apache/arrow/pull/7020#discussion_r413923773
##
File path: cpp/src/arrow/io/caching.h
##
@@ -27,6 +27,44 @@
namespace arrow {
namespace io {
+
+struct ARROW_EXPORT CacheOptions {
+ static constex
mayuropensource commented on issue #7020:
URL: https://github.com/apache/arrow/pull/7020#issuecomment-618486378
I messed up some commits. Will create a new one.
This is an automated message from the Apache Git Service.
To res
kszucs opened a new pull request #7021:
URL: https://github.com/apache/arrow/pull/7021
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
kiszk commented on a change in pull request #6985:
URL: https://github.com/apache/arrow/pull/6985#discussion_r413908372
##
File path: cpp/src/parquet/column_reader.cc
##
@@ -50,6 +51,140 @@ using arrow::internal::checked_cast;
namespace parquet {
+namespace {
+
+inline voi
github-actions[bot] commented on issue #7020:
URL: https://github.com/apache/arrow/pull/7020#issuecomment-618472360
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 al
kiszk commented on a change in pull request #6985:
URL: https://github.com/apache/arrow/pull/6985#discussion_r413905959
##
File path: cpp/src/parquet/column_reader.cc
##
@@ -50,6 +51,140 @@ using arrow::internal::checked_cast;
namespace parquet {
+namespace {
+
+inline voi
paddyhoran commented on a change in pull request #6980:
URL: https://github.com/apache/arrow/pull/6980#discussion_r413903768
##
File path: rust/arrow/src/array/builder.rs
##
@@ -236,6 +251,14 @@ impl BufferBuilderTrait for
BufferBuilder {
self.write_bytes(v.to_byte_sl
pitrou commented on issue #6846:
URL: https://github.com/apache/arrow/pull/6846#issuecomment-618464806
The CI failure looks unrelated, will merge.
This is an automated message from the Apache Git Service.
To respond to the me
mayuropensource opened a new pull request #7020:
URL: https://github.com/apache/arrow/pull/7020
JIRA: https://issues.apache.org/jira/browse/ARROW-8562
This change is not actually used until
https://github.com/apache/arrow/pull/6744 (@lidavidm) is pushed, however, it
doesn't need to
wesm commented on issue #6992:
URL: https://github.com/apache/arrow/pull/6992#issuecomment-618459883
I'm OK with this. The maintenance burden of supporting several years' worth
of pandas releases seems like a lot to bear. If there are parties which are
affected by this they should contribu
github-actions[bot] commented on issue #7019:
URL: https://github.com/apache/arrow/pull/7019#issuecomment-618456824
https://issues.apache.org/jira/browse/ARROW-8569
This is an automated message from the Apache Git Service.
To
nealrichardson commented on issue #7019:
URL: https://github.com/apache/arrow/pull/7019#issuecomment-618454886
@github-actions crossbow submit homebrew-cpp
This is an automated message from the Apache Git Service.
To respond
nealrichardson opened a new pull request #7019:
URL: https://github.com/apache/arrow/pull/7019
See https://github.com/apache/arrow/pull/6996#issuecomment-618053499
This is an automated message from the Apache Git Service.
To
bkietz commented on a change in pull request #6879:
URL: https://github.com/apache/arrow/pull/6879#discussion_r413877530
##
File path: ci/scripts/PKGBUILD
##
@@ -50,6 +52,12 @@ source_dir="$ARROW_HOME"
cpp_build_dir=build-${CARCH}-cpp
+# This should be "release" for real R
lidavidm commented on a change in pull request #6744:
URL: https://github.com/apache/arrow/pull/6744#discussion_r413864236
##
File path: cpp/src/parquet/file_reader.cc
##
@@ -536,6 +577,14 @@ std::shared_ptr
ParquetFileReader::RowGroup(int i) {
return contents_->GetRowGroup
andygrove commented on a change in pull request #7009:
URL: https://github.com/apache/arrow/pull/7009#discussion_r413857080
##
File path: rust/parquet/src/record/api.rs
##
@@ -50,6 +50,33 @@ impl Row {
pub fn len(&self) -> usize {
self.fields.len()
}
+
+p
lidavidm commented on a change in pull request #6744:
URL: https://github.com/apache/arrow/pull/6744#discussion_r413819271
##
File path: cpp/src/parquet/file_reader.cc
##
@@ -212,6 +237,21 @@ class SerializedFile : public ParquetFileReader::Contents {
file_metadata_ = std:
fsaintjacques commented on a change in pull request #6979:
URL: https://github.com/apache/arrow/pull/6979#discussion_r413811549
##
File path: cpp/src/parquet/arrow/reader.cc
##
@@ -260,12 +260,28 @@ class FileReaderImpl : public FileReader {
Status GetRecordBatchReader(con
fsaintjacques commented on a change in pull request #6979:
URL: https://github.com/apache/arrow/pull/6979#discussion_r413812220
##
File path: python/pyarrow/_parquet.pxd
##
@@ -334,7 +334,7 @@ cdef extern from "parquet/api/reader.h" namespace "parquet"
nogil:
ArrowRea
1 - 100 of 139 matches
Mail list logo