isichei commented on a change in pull request #10575:
URL: https://github.com/apache/arrow/pull/10575#discussion_r656811035
##
File path: python/pyarrow/parquet.py
##
@@ -213,15 +213,23 @@ class ParquetFile:
Coalesce and issue file reads in parallel to improve performa
github-actions[bot] commented on pull request #10575:
URL: https://github.com/apache/arrow/pull/10575#issuecomment-866580221
https://issues.apache.org/jira/browse/ARROW-13086
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
isichei opened a new pull request #10575:
URL: https://github.com/apache/arrow/pull/10575
--
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.
For queries about this service,
houqp commented on a change in pull request #571:
URL: https://github.com/apache/arrow-datafusion/pull/571#discussion_r656791849
##
File path: datafusion/src/physical_plan/planner.rs
##
@@ -283,7 +322,7 @@ impl DefaultPhysicalPlanner {
Ok(Arc::new(WindowAggExec
Dandandan commented on a change in pull request #492:
URL: https://github.com/apache/arrow-rs/pull/492#discussion_r656789000
##
File path: arrow/src/compute/kernels/concat.rs
##
@@ -57,6 +57,9 @@ pub fn concat(arrays: &[&Array]) -> Result {
return Err(ArrowError::Compu
houqp commented on a change in pull request #605:
URL: https://github.com/apache/arrow-datafusion/pull/605#discussion_r656785924
##
File path: datafusion/src/physical_plan/hash_utils.rs
##
@@ -21,25 +21,9 @@ use crate::error::{DataFusionError, Result};
use arrow::datatypes::{F
houqp opened a new pull request #605:
URL: https://github.com/apache/arrow-datafusion/pull/605
# Which issue does this PR close?
Follow up for
https://github.com/apache/arrow-datafusion/pull/55#pullrequestreview-683118433.
Closes #601.
Also fixed a bug where `index_
codecov-commenter commented on pull request #492:
URL: https://github.com/apache/arrow-rs/pull/492#issuecomment-866546109
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/492?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+A
Jimexist commented on a change in pull request #490:
URL: https://github.com/apache/arrow-rs/pull/490#discussion_r656770815
##
File path: arrow/src/record_batch.rs
##
@@ -426,6 +441,29 @@ mod tests {
assert_eq!(5, record_batch.column(1).data().len());
}
+#[t
Jimexist commented on a change in pull request #490:
URL: https://github.com/apache/arrow-rs/pull/490#discussion_r656770581
##
File path: arrow/src/record_batch.rs
##
@@ -244,6 +244,21 @@ impl RecordBatch {
&self.columns[..]
}
+/// Return a new RecordBatch w
Jimexist opened a new pull request #492:
URL: https://github.com/apache/arrow-rs/pull/492
# Which issue does this PR close?
concatenating single element array shortcut
Closes #.
# Rationale for this change
no need to any work when the given array is single element
westonpace commented on issue #10488:
URL: https://github.com/apache/arrow/issues/10488#issuecomment-866495014
@frmnboi If you can make a github repo that compiles and exhibits the error
I'd be willing to help you debug the issue.
--
This is an automated message from the Apache Git Servi
kou commented on a change in pull request #10571:
URL: https://github.com/apache/arrow/pull/10571#discussion_r656722722
##
File path: .pre-commit-config.yaml
##
@@ -29,13 +29,6 @@ repos:
entry: bash -c "git archive HEAD --prefix=apache-arrow/
--output=arrow-src.tar &&
kou commented on a change in pull request #10571:
URL: https://github.com/apache/arrow/pull/10571#discussion_r656721780
##
File path: dev/archery/archery/utils/lint.py
##
@@ -90,20 +91,75 @@ def cpp_linter(src, build_dir, clang_format=True,
cpplint=True,
class CMakeFormat
artorias1024 opened a new issue #604:
URL: https://github.com/apache/arrow-datafusion/issues/604
**Describe the bug**
When I running the DataFusion Benchmarks and enable eatures `simd`
```
artorias@artorias-ThinkPad-X1-Carbon-Gen-8:~/workspace-vscode/arrow-datafusion/benchmarks$
c
frmnboi commented on issue #10488:
URL: https://github.com/apache/arrow/issues/10488#issuecomment-866449647
To give an update, I've tried this on a second, intel-based computer and
wasn't able to get it to run without segfaulting.
As far as debugging the code goes, it claims to success
Jimexist commented on pull request #569:
URL: https://github.com/apache/arrow-datafusion/pull/569#issuecomment-866448201
> Nice, almost 5 times improvement for some queries!
>
> But do those have to do with the changes wrt sort or with regards to
partition by?
I don't expect t
lidavidm commented on pull request #10305:
URL: https://github.com/apache/arrow/pull/10305#issuecomment-866407546
Hey, sorry for the delay (again) - I can try to review this once I get a
chance, but you may also want to poke the mailing list (d...@arrow.apache.org)
to see if someone else i
codecov-commenter edited a comment on pull request #474:
URL: https://github.com/apache/arrow-rs/pull/474#issuecomment-864520421
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/474?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_ter
alamb commented on pull request #571:
URL: https://github.com/apache/arrow-datafusion/pull/571#issuecomment-866384884
I am sorry I am behind on reviews in DataFusion -- I plan to work through
the backlog tomorrow
--
This is an automated message from the Apache Git Service.
To respond to
alamb merged pull request #577:
URL: https://github.com/apache/arrow-datafusion/pull/577
--
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.
For queries about this service, p
ursabot edited a comment on pull request #10487:
URL: https://github.com/apache/arrow/pull/10487#issuecomment-866299978
Benchmark runs are scheduled for baseline =
c913aa3ad7b36b2eaccd4dc2cf0bc35ab893bb4a and contender =
bcce18e5d4d83f0831de71b363ad91470376084c. Results will be available a
lidavidm closed pull request #10573:
URL: https://github.com/apache/arrow/pull/10573
--
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.
For queries about this service, pleas
jorgecarleitao commented on issue #600:
URL:
https://github.com/apache/arrow-datafusion/issues/600#issuecomment-866337998
Perfect. I created #602 and #603 . 👍
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
jorgecarleitao opened a new issue #603:
URL: https://github.com/apache/arrow-datafusion/issues/603
Given a `StructArray` column `a` with a field `b`, I would like to be able
to run
```
SELECT a.b from t
```
returning the array associated with field `b` in the array `a`
jorgecarleitao opened a new issue #602:
URL: https://github.com/apache/arrow-datafusion/issues/602
Enabling DataFusion to support `StructArray` for some of its scalar
operations.
Note that this may be difficult to support in Ballista due to how scalar
values are currently represente
ursabot edited a comment on pull request #10487:
URL: https://github.com/apache/arrow/pull/10487#issuecomment-866299978
Benchmark runs are scheduled for baseline =
c913aa3ad7b36b2eaccd4dc2cf0bc35ab893bb4a and contender =
bcce18e5d4d83f0831de71b363ad91470376084c. Results will be available a
codecov-commenter edited a comment on pull request #488:
URL: https://github.com/apache/arrow-rs/pull/488#issuecomment-865898976
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/488?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_ter
ursabot edited a comment on pull request #10487:
URL: https://github.com/apache/arrow/pull/10487#issuecomment-866299978
Benchmark runs are scheduled for baseline =
c913aa3ad7b36b2eaccd4dc2cf0bc35ab893bb4a and contender =
bcce18e5d4d83f0831de71b363ad91470376084c. Results will be available a
nealrichardson commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656553130
##
File path: r/tests/testthat/test-dplyr-lubridate.R
##
@@ -0,0 +1,174 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
nirandaperera commented on pull request #10487:
URL: https://github.com/apache/arrow/pull/10487#issuecomment-866302771
@ursabot please benchmark command=cpp-micro
--suite-filter=arrow-compute-scalar-boolean-benchmark
--
This is an automated message from the Apache Git Service.
To respon
ursabot commented on pull request #10487:
URL: https://github.com/apache/arrow/pull/10487#issuecomment-866299978
Benchmark runs are scheduled for baseline =
c913aa3ad7b36b2eaccd4dc2cf0bc35ab893bb4a and contender =
bcce18e5d4d83f0831de71b363ad91470376084c. Results will be available as each
nirandaperera commented on pull request #10487:
URL: https://github.com/apache/arrow/pull/10487#issuecomment-866299350
@ursabot please benchmark
--
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 t
edponce commented on a change in pull request #10395:
URL: https://github.com/apache/arrow/pull/10395#discussion_r656530353
##
File path: cpp/src/arrow/compute/kernels/scalar_arithmetic.cc
##
@@ -454,6 +456,48 @@ struct PowerChecked {
}
};
+struct Sign {
+ template ::val
jorisvandenbossche commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656528973
##
File path: r/tests/testthat/test-dplyr-lubridate.R
##
@@ -0,0 +1,174 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or
edponce commented on a change in pull request #10395:
URL: https://github.com/apache/arrow/pull/10395#discussion_r656525276
##
File path: cpp/src/arrow/compute/api_scalar.cc
##
@@ -50,6 +50,11 @@ namespace compute {
SCALAR_ARITHMETIC_UNARY(AbsoluteValue, "abs", "abs_checked")
codecov-commenter edited a comment on pull request #488:
URL: https://github.com/apache/arrow-rs/pull/488#issuecomment-865898976
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/488?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_ter
lidavidm commented on pull request #10511:
URL: https://github.com/apache/arrow/pull/10511#issuecomment-866268714
Alright, I've pushed half a stab at this - I'm going to pick up ARROW-11932
so I can write a generic serialization for `vector` (where `T` itself is
serializable to Scalar). We
Dandandan commented on a change in pull request #569:
URL: https://github.com/apache/arrow-datafusion/pull/569#discussion_r656508927
##
File path: datafusion/src/physical_plan/windows.rs
##
@@ -412,11 +412,14 @@ impl ExecutionPlan for WindowAggExec {
/// Get the output p
blackary commented on pull request #10104:
URL: https://github.com/apache/arrow/pull/10104#issuecomment-866259853
Makes sense, thanks!
--
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 specifi
Dandandan commented on pull request #569:
URL: https://github.com/apache/arrow-datafusion/pull/569#issuecomment-866257274
Nice, almost 5 times improvement for some queries!
But do those have to do with the changes wrt sort or with regards to
partition by?
--
This is an automated m
alamb commented on issue #600:
URL:
https://github.com/apache/arrow-datafusion/issues/600#issuecomment-866254025
> Would you be ok for you if we add two issues, one for supporting structs
on ScalarValues, the other for supporting accessing struct fields by name on
SQL as a replacement for
lidavidm commented on pull request #10573:
URL: https://github.com/apache/arrow/pull/10573#issuecomment-866251436
Thanks Weston.
I'll merge here on green (minus the known failures).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
jorgecarleitao commented on issue #600:
URL:
https://github.com/apache/arrow-datafusion/issues/600#issuecomment-866246437
it makes a lot of sense.
Would you be ok for you if we add two issues, one for supporting structs on
ScalarValues, the other for supporting accessing struct fiel
westonpace commented on pull request #10573:
URL: https://github.com/apache/arrow/pull/10573#issuecomment-866243687
Let's leave ARROW-12364 open for the python bindings. I should be able to
get to it this week.
--
This is an automated message from the Apache Git Service.
To respond to t
alamb commented on issue #600:
URL:
https://github.com/apache/arrow-datafusion/issues/600#issuecomment-866240917
> Would supporting structs for ScalarValues solve this nicely?
I think so @jorgecarleitao - what is also needed is some way in SQL to
refer to the structs
So like
pitrou commented on a change in pull request #10574:
URL: https://github.com/apache/arrow/pull/10574#discussion_r656488996
##
File path: cpp/src/arrow/filesystem/hdfs.cc
##
@@ -134,21 +140,39 @@ class HadoopFileSystem::Impl {
}
std::vector results;
+// Fetch wor
alamb commented on pull request #55:
URL: https://github.com/apache/arrow-datafusion/pull/55#issuecomment-866238181
I filed a ticket to track some follow on work:
https://github.com/apache/arrow-datafusion/issues/601
--
This is an automated message from the Apache Git Service.
To respon
alamb edited a comment on issue #601:
URL:
https://github.com/apache/arrow-datafusion/issues/601#issuecomment-866237550
Interestingly, it seems like f2.bar is causing some issues:
```
> select f1.bar from foo as f1 JOIN foo as f2 ON f1.bar = f2.bar;
+-+
| bar |
+-
alamb commented on issue #601:
URL:
https://github.com/apache/arrow-datafusion/issues/601#issuecomment-866237550
Interestingly, it seems like f2.bar is causing some issues:
```
> select f1.bar from foo as f1 JOIN foo as f2 ON f1.bar = f2.bar;
+-+
| bar |
+-+
|
alamb opened a new issue #601:
URL: https://github.com/apache/arrow-datafusion/issues/601
**Describe the bug**
> select * from foo as f1 JOIN foo as f2 ON f1.bar = f2.bar;
thread 'main' panicked at 'index out of bounds: the len is 1 but the index
is 1',
/Users/alamb/.cargo/regist
lidavidm commented on a change in pull request #10574:
URL: https://github.com/apache/arrow/pull/10574#discussion_r656469358
##
File path: cpp/src/arrow/filesystem/test_util.cc
##
@@ -453,9 +453,11 @@ void GenericFileSystemTest::TestMoveDir(FileSystem* fs) {
AssertAllFiles(f
jorgecarleitao commented on issue #600:
URL:
https://github.com/apache/arrow-datafusion/issues/600#issuecomment-866233559
When I implemented todays' aggs, I Initially though about multiple return
values, and then concluded that the Struct is sufficient and desirable. What I
like about the
pitrou commented on pull request #10574:
URL: https://github.com/apache/arrow/pull/10574#issuecomment-866229901
Crossbow builds submitted at
https://github.com/ursacomputing/crossbow/branches/all?query=build-364
--
This is an automated message from the Apache Git Service.
To respond to t
jorgecarleitao commented on pull request #55:
URL: https://github.com/apache/arrow-datafusion/pull/55#issuecomment-866228974
Thank you @houqp for this excellent PR, together with the .md docs. They set
a really important foundation to the query's semantics 💯
--
This is an automated messa
alamb commented on pull request #55:
URL: https://github.com/apache/arrow-datafusion/pull/55#issuecomment-866222968
Thanks for the epic work @houqp -- this is really great to see
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
pitrou closed pull request #10566:
URL: https://github.com/apache/arrow/pull/10566
--
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.
For queries about this service, please
pitrou commented on a change in pull request #10395:
URL: https://github.com/apache/arrow/pull/10395#discussion_r656470969
##
File path: cpp/src/arrow/compute/api_scalar.cc
##
@@ -50,6 +50,11 @@ namespace compute {
SCALAR_ARITHMETIC_UNARY(AbsoluteValue, "abs", "abs_checked")
houqp commented on pull request #55:
URL: https://github.com/apache/arrow-datafusion/pull/55#issuecomment-866221029
Thank you all for the reviews! When I initially started working on this, I
thought it would only take me two weekends, lol. I will send the join semantic
change diff as a fol
pitrou commented on pull request #10566:
URL: https://github.com/apache/arrow/pull/10566#issuecomment-866220429
CI failures are unrelated.
--
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 spe
edponce commented on a change in pull request #10395:
URL: https://github.com/apache/arrow/pull/10395#discussion_r656468614
##
File path: cpp/src/arrow/compute/api_scalar.cc
##
@@ -50,6 +50,11 @@ namespace compute {
SCALAR_ARITHMETIC_UNARY(AbsoluteValue, "abs", "abs_checked")
pitrou commented on pull request #10574:
URL: https://github.com/apache/arrow/pull/10574#issuecomment-866220006
@github-actions crossbow submit test-conda-python-3.7-hdfs-3.2.1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
pitrou commented on pull request #10574:
URL: https://github.com/apache/arrow/pull/10574#issuecomment-866219885
@github-actions crossbow submit test-conda-python-3.7-hdfs-2.9.2
test-conda-python-3.7-hdfs-3.21
--
This is an automated message from the Apache Git Service.
To respond to the
pitrou commented on pull request #10574:
URL: https://github.com/apache/arrow/pull/10574#issuecomment-866217633
@github-actions crossbow submit test-*hdfs*
--
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
Dandandan commented on pull request #55:
URL: https://github.com/apache/arrow-datafusion/pull/55#issuecomment-866217530
Woohoo 🎉 🎉🎉🎉
--
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 spec
github-actions[bot] commented on pull request #10574:
URL: https://github.com/apache/arrow/pull/10574#issuecomment-866215494
https://issues.apache.org/jira/browse/ARROW-12790
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
pitrou opened a new pull request #10574:
URL: https://github.com/apache/arrow/pull/10574
* Ensure the HadoopFileSystem meets most requirements from the FileSystem
API.
* Implement HadoopFileSystem::CopyFile.
* Enable generic filesystem tests for HadoopFileSystem.
* Add generic file
alamb closed issue #56:
URL: https://github.com/apache/arrow-datafusion/issues/56
--
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.
For queries about this service, please c
alamb closed issue #57:
URL: https://github.com/apache/arrow-datafusion/issues/57
--
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.
For queries about this service, please c
alamb closed issue #311:
URL: https://github.com/apache/arrow-datafusion/issues/311
--
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.
For queries about this service, please
alamb merged pull request #55:
URL: https://github.com/apache/arrow-datafusion/pull/55
--
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.
For queries about this service, ple
alamb commented on pull request #55:
URL: https://github.com/apache/arrow-datafusion/pull/55#issuecomment-866215121
🚀
--
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.
For
alamb commented on a change in pull request #474:
URL: https://github.com/apache/arrow-rs/pull/474#discussion_r656455542
##
File path: arrow/src/array/array_list.rs
##
@@ -50,6 +50,9 @@ impl OffsetSizeTrait for i64 {
}
}
+/// Generic struct for a primitive Array
+///
+/
edponce commented on a change in pull request #10395:
URL: https://github.com/apache/arrow/pull/10395#discussion_r656452847
##
File path: cpp/src/arrow/compute/kernels/scalar_arithmetic_test.cc
##
@@ -149,7 +173,23 @@ template
class TestUnaryArithmeticUnsigned : public TestUn
edponce commented on a change in pull request #10395:
URL: https://github.com/apache/arrow/pull/10395#discussion_r656451863
##
File path: cpp/src/arrow/compute/api_scalar.cc
##
@@ -50,6 +50,11 @@ namespace compute {
SCALAR_ARITHMETIC_UNARY(AbsoluteValue, "abs", "abs_checked")
nealrichardson commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656451946
##
File path: r/tests/testthat/test-dplyr-lubridate.R
##
@@ -0,0 +1,174 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
nealrichardson commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656451510
##
File path: r/tests/testthat/test-dplyr-lubridate.R
##
@@ -0,0 +1,174 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
nealrichardson commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656451343
##
File path: r/tests/testthat/test-dplyr-lubridate.R
##
@@ -0,0 +1,174 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
nealrichardson commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656449850
##
File path: r/R/expression.R
##
@@ -29,8 +29,17 @@
# stringr spellings of those
"str_length" = "utf8_length",
"str_to_lower" = "utf8_low
nealrichardson commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656449431
##
File path: r/R/dplyr-functions.R
##
@@ -442,3 +442,74 @@ nse_funcs$strptime <- function(x, format = "%Y-%m-%d
%H:%M:%S", tz = NULL, unit
E
nealrichardson commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656449141
##
File path: r/R/dplyr-functions.R
##
@@ -442,3 +442,74 @@ nse_funcs$strptime <- function(x, format = "%Y-%m-%d
%H:%M:%S", tz = NULL, unit
E
nealrichardson commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656448695
##
File path: r/R/dplyr-functions.R
##
@@ -442,3 +442,74 @@ nse_funcs$strptime <- function(x, format = "%Y-%m-%d
%H:%M:%S", tz = NULL, unit
E
nealrichardson commented on a change in pull request #10507:
URL: https://github.com/apache/arrow/pull/10507#discussion_r656447630
##
File path: r/R/dplyr-functions.R
##
@@ -442,3 +442,37 @@ nse_funcs$strptime <- function(x, format = "%Y-%m-%d
%H:%M:%S", tz = NULL, unit
E
alamb opened a new issue #600:
URL: https://github.com/apache/arrow-datafusion/issues/600
# Usecase
I want to implement a user defined aggregate function that produces more
than one column ( logical values)
Specifically I am trying to implement the InfluxDB 'selector' functions
`
github-actions[bot] commented on pull request #10573:
URL: https://github.com/apache/arrow/pull/10573#issuecomment-866187741
https://issues.apache.org/jira/browse/ARROW-10440
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
bkietz opened a new pull request #10573:
URL: https://github.com/apache/arrow/pull/10573
This enables collection of paths written to during writing of a
FileSystemDataset
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
b41sh commented on a change in pull request #490:
URL: https://github.com/apache/arrow-rs/pull/490#discussion_r656403180
##
File path: arrow/src/record_batch.rs
##
@@ -244,6 +244,18 @@ impl RecordBatch {
&self.columns[..]
}
+/// Return a new RecordBatch wher
b41sh commented on a change in pull request #490:
URL: https://github.com/apache/arrow-rs/pull/490#discussion_r656393219
##
File path: arrow/src/record_batch.rs
##
@@ -244,6 +244,18 @@ impl RecordBatch {
&self.columns[..]
}
+/// Return a new RecordBatch wher
nevi-me commented on a change in pull request #377:
URL: https://github.com/apache/arrow-rs/pull/377#discussion_r654833725
##
File path: arrow-flight/src/utils.rs
##
@@ -164,4 +73,40 @@ pub fn flight_data_to_arrow_batch(
})?
}
-// TODO: add more explicit conversion
nevi-me commented on a change in pull request #490:
URL: https://github.com/apache/arrow-rs/pull/490#discussion_r656390969
##
File path: arrow/src/record_batch.rs
##
@@ -244,6 +244,18 @@ impl RecordBatch {
&self.columns[..]
}
+/// Return a new RecordBatch wh
jorgecarleitao commented on a change in pull request #490:
URL: https://github.com/apache/arrow-rs/pull/490#discussion_r656382277
##
File path: arrow/src/record_batch.rs
##
@@ -244,6 +244,18 @@ impl RecordBatch {
&self.columns[..]
}
+/// Return a new RecordB
nevi-me opened a new pull request #491:
URL: https://github.com/apache/arrow-rs/pull/491
# Which issue does this PR close?
Related #395 .
# Rationale for this change
To add support for the map array :)
# What changes are included in this PR?
This adds mini
b41sh opened a new pull request #490:
URL: https://github.com/apache/arrow-rs/pull/490
# Which issue does this PR close?
Closes #460
# Rationale for this change
`slice` can be used to handle part of `RecordBatch`
# What changes are included in this P
westonpace commented on pull request #10104:
URL: https://github.com/apache/arrow/pull/10104#issuecomment-866122199
Yes. Patch releases are focused on regressions / major errors. 4.0.1 was
created by cherry-picking select commits on top of 4.0.0 which did not include
this fix. This chan
kszucs commented on a change in pull request #10572:
URL: https://github.com/apache/arrow/pull/10572#discussion_r656363881
##
File path: .github/workflows/python.yml
##
@@ -140,12 +140,6 @@ jobs:
- name: Fetch Submodules and Tags
shell: bash
run: ci/scr
github-actions[bot] commented on pull request #10572:
URL: https://github.com/apache/arrow/pull/10572#issuecomment-866113411
https://issues.apache.org/jira/browse/ARROW-6513
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
kszucs opened a new pull request #10572:
URL: https://github.com/apache/arrow/pull/10572
--
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.
For queries about this service, p
kszucs closed pull request #10556:
URL: https://github.com/apache/arrow/pull/10556
--
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.
For queries about this service, please
kszucs commented on pull request #10556:
URL: https://github.com/apache/arrow/pull/10556#issuecomment-866103992
The build failures are unrelated, merging.
--
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
kszucs closed pull request #10570:
URL: https://github.com/apache/arrow/pull/10570
--
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.
For queries about this service, please
1 - 100 of 282 matches
Mail list logo
