viirya commented on code in PR #1871:
URL: https://github.com/apache/arrow-rs/pull/1871#discussion_r896426518
##
arrow/src/util/decimal.rs:
##
@@ -0,0 +1,148 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NO
viirya commented on code in PR #1871:
URL: https://github.com/apache/arrow-rs/pull/1871#discussion_r896426518
##
arrow/src/util/decimal.rs:
##
@@ -0,0 +1,148 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NO
codecov-commenter commented on PR #1871:
URL: https://github.com/apache/arrow-rs/pull/1871#issuecomment-1154757340
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1871?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
mingmwang commented on issue #1972:
URL:
https://github.com/apache/arrow-datafusion/issues/1972#issuecomment-1154749123
Nice discussion!
I see many people raised the points to build the volcano/cascades style
planner for DataFusion, either on Egg or to build from scratch. To implemen
viirya commented on code in PR #1871:
URL: https://github.com/apache/arrow-rs/pull/1871#discussion_r896400715
##
arrow/src/util/decimal.rs:
##
@@ -0,0 +1,148 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NO
wjones127 commented on code in PR #12775:
URL: https://github.com/apache/arrow/pull/12775#discussion_r896399905
##
cpp/examples/arrow/rapidjson_row_converter.cc:
##
@@ -0,0 +1,590 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license a
viirya opened a new pull request, #1871:
URL: https://github.com/apache/arrow-rs/pull/1871
# Which issue does this PR close?
Closes #1870.
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing chan
viirya opened a new issue, #1870:
URL: https://github.com/apache/arrow-rs/issues/1870
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
Part of #131.
Currently we directly use i128 in decimal related APIs, for example
Dec
github-actions[bot] commented on PR #13371:
URL: https://github.com/apache/arrow/pull/13371#issuecomment-1154726650
Revision: d5aa1d781334242606f564e266e8de0c6029eaa9
Submitted crossbow builds: [ursacomputing/crossbow @
actions-86055eb4d2](https://github.com/ursacomputing/crossbow/bra
kou commented on PR #13371:
URL: https://github.com/apache/arrow/pull/13371#issuecomment-1154725762
@github-actions crossbow submit almalinux-* centos-* amazon-linux-*
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
dependabot[bot] commented on PR #70:
URL: https://github.com/apache/arrow-ballista/pull/70#issuecomment-1154711395
The following labels could not be found: `auto-dependencies`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
dependabot[bot] opened a new pull request, #70:
URL: https://github.com/apache/arrow-ballista/pull/70
Updates the requirements on [arrow](https://github.com/apache/arrow-rs) to
permit the latest version.
Changelog
Sourced from https://github.com/apache/arrow-rs/blob/master/CHANGELO
dependabot[bot] commented on PR #69:
URL: https://github.com/apache/arrow-ballista/pull/69#issuecomment-1154710733
The following labels could not be found: `auto-dependencies`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
dependabot[bot] opened a new pull request, #69:
URL: https://github.com/apache/arrow-ballista/pull/69
Updates the requirements on
[arrow-flight](https://github.com/apache/arrow-rs) to permit the latest version.
Changelog
Sourced from https://github.com/apache/arrow-rs/blob/master/C
eerhardt merged PR #13279:
URL: https://github.com/apache/arrow/pull/13279
--
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: github-unsubscr...@arrow.apac
andygrove closed pull request #2728: Deprecate `datafusion::logical_plan` module
URL: https://github.com/apache/arrow-datafusion/pull/2728
--
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
andygrove commented on PR #66:
URL: https://github.com/apache/arrow-ballista/pull/66#issuecomment-1154662650
Unfortunately we cannot use DF 9.0.0 due to a breaking API change. I created
https://github.com/apache/arrow-ballista/pull/68 to upgrade to the latest
version which has the fix the t
andygrove opened a new pull request, #68:
URL: https://github.com/apache/arrow-ballista/pull/68
# Which issue does this PR close?
N/A
# Rationale for this change
Stay up to date
# What changes are included in this PR?
Bump DF version
github-actions[bot] commented on PR #13218:
URL: https://github.com/apache/arrow/pull/13218#issuecomment-1154658694
https://issues.apache.org/jira/browse/ARROW-16590
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
wesm merged PR #13218:
URL: https://github.com/apache/arrow/pull/13218
--
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: github-unsubscr...@arrow.apache.o
AssHero commented on PR #2721:
URL:
https://github.com/apache/arrow-datafusion/pull/2721#issuecomment-1154648956
> Please let me know if I can provide some info you need🧐
Do you have any ideas about the test failure? I can not reproduce that case.
--
This is an automated message fr
yahoNanJing commented on PR #59:
URL: https://github.com/apache/arrow-ballista/pull/59#issuecomment-1154642089
Hi @thinkharderdev, just left a few 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
rishabh-rana commented on PR #13279:
URL: https://github.com/apache/arrow/pull/13279#issuecomment-1154641991
@eerhardt Thank you for your review! All good to merge now
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
Ted-Jiang commented on PR #2726:
URL:
https://github.com/apache/arrow-datafusion/pull/2726#issuecomment-1154638607
> Thanks @Ted-Jiang
>
> You probably know it, but you can also use `EXPLAIN VERBOSE` to see the
output of the plans after each pass
>
> For example:
>
> ``
Ted-Jiang commented on issue #2725:
URL:
https://github.com/apache/arrow-datafusion/issues/2725#issuecomment-1154631256
> If I read your conclusion corrctly, I think you are saying that the filter
added by the filter pushdown rule is remapping the names somehow. Specifically,
you are propo
codecov-commenter commented on PR #2728:
URL:
https://github.com/apache/arrow-datafusion/pull/2728#issuecomment-1154630207
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/2728?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_
avertleyb commented on code in PR #13376:
URL: https://github.com/apache/arrow/pull/13376#discussion_r896300314
##
cpp/src/arrow/util/bit_util.h:
##
@@ -67,7 +67,14 @@ static constexpr uint8_t kBytePopcount[] = {
5, 4, 5, 5, 6, 3, 4, 4, 5, 4, 5, 5, 6, 4, 5, 5, 6, 5, 6, 6, 7
andygrove opened a new pull request, #2728:
URL: https://github.com/apache/arrow-datafusion/pull/2728
# Which issue does this PR close?
Closes #.
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing change
kou merged PR #13374:
URL: https://github.com/apache/arrow/pull/13374
--
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: github-unsubscr...@arrow.apache.or
kou commented on PR #13374:
URL: https://github.com/apache/arrow/pull/13374#issuecomment-1154584199
+1
--
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-mai
kou commented on issue #1114:
URL:
https://github.com/apache/arrow-datafusion/issues/1114#issuecomment-1154572691
@andygrove Could you create
https://github.com/datafusion-contrib/datafusion-ruby and give me admin
permission? We can implement Ruby bindings based on
https://github.com/data
andygrove commented on PR #67:
URL: https://github.com/apache/arrow-ballista/pull/67#issuecomment-1154569407
@nl5887 Do you have examples of panics that this fixes? Can we fix the root
cause by using `Result` instead of calling `panic!` or `unimplemented!`?
--
This is an automated message
github-actions[bot] commented on PR #13371:
URL: https://github.com/apache/arrow/pull/13371#issuecomment-1154569182
Revision: e2dedd560fec0e74100fd33aa4bbc387ae12048b
Submitted crossbow builds: [ursacomputing/crossbow @
actions-2595c22265](https://github.com/ursacomputing/crossbow/bra
kou commented on PR #13371:
URL: https://github.com/apache/arrow/pull/13371#issuecomment-1154568558
@github-actions crossbow submit almalinux-* centos-* amazon-linux-*
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
andygrove merged PR #2715:
URL: https://github.com/apache/arrow-datafusion/pull/2715
--
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: github-unsubscr...@
andygrove closed issue #2712: Common Subexpression Eliminiation pass errors if
run twice on some plans: Schema contains duplicate unqualified field name
'IsNull-Column-sys.host'
URL: https://github.com/apache/arrow-datafusion/issues/2712
--
This is an automated message from the Apache Git Se
andygrove opened a new pull request, #2727:
URL: https://github.com/apache/arrow-datafusion/pull/2727
# Which issue does this PR close?
N/A
# Rationale for this change
The instructions for releasing the crates was incorrect
# What changes are included
nevi-me commented on issue #1858:
URL: https://github.com/apache/arrow-rs/issues/1858#issuecomment-1154517263
Comparison from the last time this work was done
https://github.com/apache/arrow-rs/pull/715#issuecomment-917619317
--
This is an automated message from the Apache Git Service.
To
viirya commented on PR #1848:
URL: https://github.com/apache/arrow-rs/pull/1848#issuecomment-1154506626
Thanks @sunchao !
--
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
viirya closed issue #1847: PyArrow intergation test for C Stream Interface
URL: https://github.com/apache/arrow-rs/issues/1847
--
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.
T
viirya merged PR #1848:
URL: https://github.com/apache/arrow-rs/pull/1848
--
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: github-unsubscr...@arrow.apach
codecov-commenter commented on PR #1865:
URL: https://github.com/apache/arrow-rs/pull/1865#issuecomment-1154480873
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1865?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
codecov-commenter commented on PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#issuecomment-1154477386
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1866?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
tustvold merged PR #1869:
URL: https://github.com/apache/arrow-rs/pull/1869
--
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: github-unsubscr...@arrow.apa
kou commented on code in PR #13376:
URL: https://github.com/apache/arrow/pull/13376#discussion_r896152027
##
cpp/src/arrow/util/bit_util.h:
##
@@ -67,7 +67,14 @@ static constexpr uint8_t kBytePopcount[] = {
5, 4, 5, 5, 6, 3, 4, 4, 5, 4, 5, 5, 6, 4, 5, 5, 6, 5, 6, 6, 7, 3, 4
tustvold closed issue #1862: Exclude some long-running tests from miri
URL: https://github.com/apache/arrow-rs/issues/1862
--
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 un
tustvold merged PR #1863:
URL: https://github.com/apache/arrow-rs/pull/1863
--
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: github-unsubscr...@arrow.apa
rtpsw commented on code in PR #13375:
URL: https://github.com/apache/arrow/pull/13375#discussion_r896138388
##
cpp/src/arrow/engine/substrait/serde.cc:
##
@@ -58,12 +58,53 @@ Result DeserializeRelation(const
Buffer& buf,
return FromProto(rel, ext_set);
}
-Result> Deserial
jhorstmann commented on code in PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#discussion_r896126605
##
arrow/src/buffer/immutable.rs:
##
@@ -181,19 +181,14 @@ impl Buffer {
/// View buffer as typed slice.
///
-/// # Safety
+/// # Panics
///
kou commented on PR #13244:
URL: https://github.com/apache/arrow/pull/13244#issuecomment-1154423528
`docker build` didn't use pulled images as cache as @kszucs said.
I added the following workflow as a workaround (see the description of this
pull request for details, I've updated):
tustvold commented on code in PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#discussion_r896113066
##
arrow/src/buffer/immutable.rs:
##
@@ -181,19 +181,14 @@ impl Buffer {
/// View buffer as typed slice.
///
-/// # Safety
+/// # Panics
///
-
tustvold commented on code in PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#discussion_r896113066
##
arrow/src/buffer/immutable.rs:
##
@@ -181,19 +181,14 @@ impl Buffer {
/// View buffer as typed slice.
///
-/// # Safety
+/// # Panics
///
-
tustvold commented on code in PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#discussion_r896113066
##
arrow/src/buffer/immutable.rs:
##
@@ -181,19 +181,14 @@ impl Buffer {
/// View buffer as typed slice.
///
-/// # Safety
+/// # Panics
///
-
iChauster commented on code in PR #13028:
URL: https://github.com/apache/arrow/pull/13028#discussion_r896110522
##
cpp/src/arrow/compute/exec/asof_join_node.cc:
##
@@ -0,0 +1,806 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license ag
github-actions[bot] commented on PR #13376:
URL: https://github.com/apache/arrow/pull/13376#issuecomment-1154394087
:warning: Ticket **has not been started in JIRA**, please click 'Start
Progress'.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
github-actions[bot] commented on PR #13376:
URL: https://github.com/apache/arrow/pull/13376#issuecomment-1154394062
https://issues.apache.org/jira/browse/ARROW-16778
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
alamb commented on code in PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#discussion_r896100716
##
arrow/src/buffer/immutable.rs:
##
@@ -181,19 +181,14 @@ impl Buffer {
/// View buffer as typed slice.
///
-/// # Safety
+/// # Panics
///
-
alamb commented on issue #1327:
URL:
https://github.com/apache/arrow-datafusion/issues/1327#issuecomment-1154387746
FYI @thinkharderdev bashed this out in
https://github.com/apache/arrow-datafusion/pull/2716 🥇
--
This is an automated message from the Apache Git Service.
To respond to t
alamb commented on issue #2725:
URL:
https://github.com/apache/arrow-datafusion/issues/2725#issuecomment-1154384486
Hi @Ted-Jiang
If I read your conclusion corrctly, I think you are saying that the filter
added by the filter pushdown rule is remapping the names somehow. Specifically
MazterQyou commented on code in PR #1846:
URL: https://github.com/apache/arrow-rs/pull/1846#discussion_r896095995
##
arrow/src/compute/kernels/comparison.rs:
##
@@ -548,6 +548,89 @@ pub fn ilike_utf8_scalar(
Ok(BooleanArray::from(data))
}
+/// Perform SQL `left NOT ILIKE
wesm commented on code in PR #13218:
URL: https://github.com/apache/arrow/pull/13218#discussion_r896088311
##
cpp/src/arrow/compute/row/row_internal.h:
##
@@ -0,0 +1,250 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
wesm commented on code in PR #13218:
URL: https://github.com/apache/arrow/pull/13218#discussion_r896084320
##
cpp/src/arrow/compute/row/row_internal.h:
##
@@ -0,0 +1,250 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
alamb merged PR #2716:
URL: https://github.com/apache/arrow-datafusion/pull/2716
--
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: github-unsubscr...@arro
alamb closed issue #1327: implement grouping sets, cubes, and rollups
URL: https://github.com/apache/arrow-datafusion/issues/1327
--
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.
alamb commented on PR #2716:
URL:
https://github.com/apache/arrow-datafusion/pull/2716#issuecomment-1154368725
Looks great to me -- thanks again for all the work @thinkharderdev 🎉
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
github-actions[bot] commented on PR #13376:
URL: https://github.com/apache/arrow/pull/13376#issuecomment-1154366224
Thanks for opening a pull request!
If this is not a [minor
PR](https://github.com/apache/arrow/blob/master/CONTRIBUTING.md#Minor-Fixes).
Could you open an issue
alamb commented on code in PR #2718:
URL: https://github.com/apache/arrow-datafusion/pull/2718#discussion_r896076556
##
datafusion/core/src/physical_plan/planner.rs:
##
@@ -1544,20 +1540,22 @@ mod tests {
.build()?;
let execution_plan = plan(&logical_plan).
tustvold merged PR #1868:
URL: https://github.com/apache/arrow-rs/pull/1868
--
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: github-unsubscr...@arrow.apa
codecov-commenter commented on PR #1863:
URL: https://github.com/apache/arrow-rs/pull/1863#issuecomment-1154260033
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1863?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
nevi-me merged PR #1859:
URL: https://github.com/apache/arrow-rs/pull/1859
--
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: github-unsubscr...@arrow.apac
nevi-me closed issue #1856: PrimitiveArray::from_iter should omit validity
buffer if all values are valid
URL: https://github.com/apache/arrow-rs/issues/1856
--
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 t
jhorstmann commented on PR #1863:
URL: https://github.com/apache/arrow-rs/pull/1863#issuecomment-1154245280
Disabling stacked-borrow checking is not perfect, but probably better to run
miri with one less check than not running it at all. The check using a lot of
cpu and memory is a known pr
thinkharderdev commented on PR #59:
URL: https://github.com/apache/arrow-ballista/pull/59#issuecomment-1154240883
@yahoNanJing Made a small change in the event loop. It will now eagerly
attempt to schedule additional pending tasks for a job on update. I think this
should address your point
ursabot commented on PR #13357:
URL: https://github.com/apache/arrow/pull/13357#issuecomment-1154227721
['Python', 'R'] benchmarks have high level of regressions.
[ursa-i9-9960x](https://conbench.ursa.dev/compare/runs/2d43fcebf510411f9ef33f0341b63fca...843eadbe9ebf4bd7a23850d37a70db8e/)
ursabot commented on PR #13357:
URL: https://github.com/apache/arrow/pull/13357#issuecomment-1154227585
Benchmark runs are scheduled for baseline =
a53f2bda5dfa50f2d1dbfd9d7fbc98d8c9725564 and contender =
c5a74802b8e02ba882ff09393b8e1afd9d28270f.
c5a74802b8e02ba882ff09393b8e1afd9d28270f is
tustvold commented on issue #1867:
URL: https://github.com/apache/arrow-rs/issues/1867#issuecomment-1154211295
I've run out of time to work on this, but #1868 should get the pipes moving
again
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
codecov-commenter commented on PR #1869:
URL: https://github.com/apache/arrow-rs/pull/1869#issuecomment-1154210706
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1869?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
tustvold commented on PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#issuecomment-1154210410
Clippy fix in #1868
--
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
codecov-commenter commented on PR #1868:
URL: https://github.com/apache/arrow-rs/pull/1868#issuecomment-1154202292
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1868?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
viirya commented on issue #131:
URL: https://github.com/apache/arrow-rs/issues/131#issuecomment-1154197732
I think you mean to make Decimal128 to be compatible with i128 so it won't
break existing user code?
It makes sense to me.
--
This is an automated message from the Apache Git
tustvold opened a new pull request, #1869:
URL: https://github.com/apache/arrow-rs/pull/1869
Running
```
rm Cargo.lock
cargo check
```
Results in the following changes to the generated files. It also generates
an empty protobuf file which appears to relate to
https://
tustvold opened a new pull request, #1868:
URL: https://github.com/apache/arrow-rs/pull/1868
# Which issue does this PR close?
Part of #1867
# Rationale for this change
This gets the pipes moving again, whilst working on trying to understand
what has been changed...
MazterQyou commented on code in PR #1846:
URL: https://github.com/apache/arrow-rs/pull/1846#discussion_r895954685
##
arrow/src/compute/kernels/comparison.rs:
##
@@ -548,6 +548,89 @@ pub fn ilike_utf8_scalar(
Ok(BooleanArray::from(data))
}
+/// Perform SQL `left NOT ILIKE
tustvold opened a new issue, #1867:
URL: https://github.com/apache/arrow-rs/issues/1867
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
The most recent clap release deprecated some functionality which is causing
clippy to fail
j-a-m-l commented on issue #1114:
URL:
https://github.com/apache/arrow-datafusion/issues/1114#issuecomment-1154163105
Just to clarify: I can't be involved in this binding, since my current
employer is not interested and I don't have enough free time to do it.
--
This is an automated mess
tustvold commented on code in PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#discussion_r895931569
##
arrow/src/buffer/mutable.rs:
##
@@ -299,7 +296,7 @@ impl MutableBuffer {
/// assert_eq!(buffer.len(), 8) // u32 has 4 bytes
/// ```
#[inline]
-pu
tustvold commented on code in PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#discussion_r895935063
##
parquet/src/arrow/arrow_writer/mod.rs:
##
@@ -576,7 +576,7 @@ macro_rules! def_get_binary_array_fn {
fn $name(array: &$ty) -> Vec {
let mut by
tustvold commented on code in PR #1866:
URL: https://github.com/apache/arrow-rs/pull/1866#discussion_r895931569
##
arrow/src/buffer/mutable.rs:
##
@@ -299,7 +296,7 @@ impl MutableBuffer {
/// assert_eq!(buffer.len(), 8) // u32 has 4 bytes
/// ```
#[inline]
-pu
vibhatha commented on code in PR #13375:
URL: https://github.com/apache/arrow/pull/13375#discussion_r895930793
##
cpp/src/arrow/engine/substrait/serde.cc:
##
@@ -58,12 +58,53 @@ Result DeserializeRelation(const
Buffer& buf,
return FromProto(rel, ext_set);
}
-Result> Deser
tustvold opened a new pull request, #1866:
URL: https://github.com/apache/arrow-rs/pull/1866
# Which issue does this PR close?
Closes #996
Closes #1027
# Rationale for this change
These APIs were originally marked unsafe because `ArrowNativeType` could be
implemente
dominicbarnes commented on PR #13322:
URL: https://github.com/apache/arrow/pull/13322#issuecomment-1154147685
@zeroshade thanks for getting this merged! How can I help getting the fix
backported?
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
assignUser commented on PR #13244:
URL: https://github.com/apache/arrow/pull/13244#issuecomment-1154145310
2h 51min vs 1h 🥳 🚀
--
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
vibhatha commented on code in PR #13375:
URL: https://github.com/apache/arrow/pull/13375#discussion_r895922931
##
cpp/src/arrow/engine/substrait/plan_internal.cc:
##
@@ -92,6 +92,9 @@ Status AddExtensionSetToPlan(const ExtensionSet& ext_set,
substrait::Plan* plan)
Result Get
emkornfield commented on code in PR #12775:
URL: https://github.com/apache/arrow/pull/12775#discussion_r895891829
##
cpp/examples/arrow/rapidjson_row_converter.cc:
##
@@ -0,0 +1,590 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license
wesm merged PR #13364:
URL: https://github.com/apache/arrow/pull/13364
--
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: github-unsubscr...@arrow.apache.o
wesm commented on PR #13364:
URL: https://github.com/apache/arrow/pull/13364#issuecomment-1154109376
I started working on refactoring the remaining kernels -- any further
review/comments here I will address in follow up PRs or you can add new issues
linked to the parent issue ARROW-16755
emkornfield commented on code in PR #12775:
URL: https://github.com/apache/arrow/pull/12775#discussion_r895888362
##
cpp/examples/arrow/rapidjson_row_converter.cc:
##
@@ -0,0 +1,590 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license
rtpsw commented on code in PR #13375:
URL: https://github.com/apache/arrow/pull/13375#discussion_r89586
##
cpp/src/arrow/engine/substrait/plan_internal.cc:
##
@@ -92,6 +92,9 @@ Status AddExtensionSetToPlan(const ExtensionSet& ext_set,
substrait::Plan* plan)
Result GetExt
thinkharderdev commented on PR #59:
URL: https://github.com/apache/arrow-ballista/pull/59#issuecomment-1154063171
> Hi @thinkharderdev,
>
> > On this point I disagree. High availability is one goal of this work but
another is horizontal scalability. The work involved in physical plann
vibhatha commented on code in PR #13375:
URL: https://github.com/apache/arrow/pull/13375#discussion_r895853513
##
cpp/src/arrow/engine/substrait/plan_internal.cc:
##
@@ -92,6 +92,9 @@ Status AddExtensionSetToPlan(const ExtensionSet& ext_set,
substrait::Plan* plan)
Result Get
1 - 100 of 183 matches
Mail list logo
