tustvold commented on issue #1882:
URL: https://github.com/apache/arrow-rs/issues/1882#issuecomment-1159378665
A new release has been cut which updates some dependencies, including
hashbrown https://github.com/bluss/indexmap/pull/231
--
This is an automated message from the Apache Git Ser
tustvold commented on issue #1886:
URL: https://github.com/apache/arrow-rs/issues/1886#issuecomment-1159371207
Hi, I'm not very familiar with parquet-mr which your example appears to be
based on, nor am I hugely knowledgeable about the record APIs for reading
parquet, but I'll try to help o
tustvold commented on PR #1890:
URL: https://github.com/apache/arrow-rs/pull/1890#issuecomment-1159369988
I think a rebase should clear the test failures related to IPC dictionaries
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
save-buffer commented on PR #13364:
URL: https://github.com/apache/arrow/pull/13364#issuecomment-1159369450
Yes, that was my experience as well. When looking with Apple's TimeProfiler
I saw a ton of 0.1% to 1% stack traces. The execution really is spread out such
that it's hard to pinpoint
tustvold commented on PR #1900:
URL: https://github.com/apache/arrow-rs/pull/1900#issuecomment-1159369179
Integration test failures seem unrelated, so getting this one in.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
tustvold merged PR #1900:
URL: https://github.com/apache/arrow-rs/pull/1900
--
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
tustvold closed issue #1899: Final slicing in combine_option_bitmap needs to
use bit slices
URL: https://github.com/apache/arrow-rs/issues/1899
--
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 sp
tustvold merged PR #1893:
URL: https://github.com/apache/arrow-rs/pull/1893
--
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
tustvold closed issue #1892: Dictionary IPC writer appears to write incorrect
schema
URL: https://github.com/apache/arrow-rs/issues/1892
--
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
nevi-me commented on issue #1642:
URL: https://github.com/apache/arrow-rs/issues/1642#issuecomment-1159366876
@tustvold here's Arrow's spec:
https://github.com/apache/arrow/blob/master/format/Schema.fbs#L103-L131
```rust
/// A Map is a logical nested type that is represented as
tustvold commented on issue #1642:
URL: https://github.com/apache/arrow-rs/issues/1642#issuecomment-1159362939
That is my understanding, but some spelunking in the C++ or Java
implementations may be warranted to confirm how they choose to handle it
--
This is an automated message from the
tustvold commented on issue #1699:
URL: https://github.com/apache/arrow-rs/issues/1699#issuecomment-1159362165
Hi @frolovdev. I'm afraid I'm away from a computer for the next few days,
but taking a cursory look, lines like
```
let entry_len = rep_levels.iter().filter(|level
liukun4515 commented on issue #13391:
URL: https://github.com/apache/arrow/issues/13391#issuecomment-1159354832
> They're tested by the integration suite, e.g. see the output of this run:
https://github.com/apache/arrow/runs/6927219399?check_suite_focus=true#step:7:11104
thank you and
marsupialtail commented on code in PR #13385:
URL: https://github.com/apache/arrow/pull/13385#discussion_r900691273
##
cpp/src/arrow/filesystem/s3fs.h:
##
@@ -102,6 +102,8 @@ struct ARROW_EXPORT S3Options {
/// the region (environment variables, configuration profile, EC2 met
codecov-commenter commented on PR #2741:
URL:
https://github.com/apache/arrow-datafusion/pull/2741#issuecomment-1159351402
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/2741?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_
westonpace merged PR #13252:
URL: https://github.com/apache/arrow/pull/13252
--
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.ap
westonpace commented on PR #13364:
URL: https://github.com/apache/arrow/pull/13364#issuecomment-1159348973
I spent some time playing around with the
`ExecuteScalarExpressionOverhead/complex_expression` benchmark today to see if
this PR made much change there. I didn't see too much change (
xudong963 merged PR #2745:
URL: https://github.com/apache/arrow-datafusion/pull/2745
--
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...@
xudong963 commented on code in PR #2741:
URL: https://github.com/apache/arrow-datafusion/pull/2741#discussion_r900687864
##
benchmarks/tpch-gen.sh:
##
@@ -21,7 +21,12 @@
pushd ..
. ./dev/build-set-env.sh
popd
-docker build -t datafusion-tpchgen:$DATAFUSION_VERSION -f tpchgen.
xudong963 commented on code in PR #2741:
URL: https://github.com/apache/arrow-datafusion/pull/2741#discussion_r900635837
##
benchmarks/tpch-gen.sh:
##
@@ -21,7 +21,12 @@
pushd ..
. ./dev/build-set-env.sh
popd
-docker build -t datafusion-tpchgen:$DATAFUSION_VERSION -f tpchgen.
wesm commented on code in PR #13398:
URL: https://github.com/apache/arrow/pull/13398#discussion_r900630977
##
cpp/src/arrow/compute/row/grouper.cc:
##
@@ -119,7 +125,13 @@ struct GrouperImpl : Grouper {
}
for (int i = 0; i < batch.num_values(); ++i) {
- RETURN_N
wesm opened a new pull request, #13398:
URL: https://github.com/apache/arrow/pull/13398
This is mostly mechanical refactoring. Since many VectorKernels support
being passed in a ChunkedArray, I separated the `ExecSpan` code path (which
does not support chunked arrays) from a separate `Vecto
github-actions[bot] commented on PR #13398:
URL: https://github.com/apache/arrow/pull/13398#issuecomment-1159325954
https://issues.apache.org/jira/browse/ARROW-16824
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
wesm commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159285230
Yes, definitely. I'm just referring to the _implementation_ of e.g.
`f(scalar) -> scalar)` or `g(scalar, scalar) -> scalar` — the ability to
perform these operations will remain but the impl
westonpace commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159283611
I think I see your point. Is the code path for `array x scalar` remaining
(e.g. `add(field_ref("x"), 7)`)?
--
This is an automated message from the Apache Git Service.
To respond to
dongjoon-hyun commented on PR #13392:
URL: https://github.com/apache/arrow/pull/13392#issuecomment-1159281615
Apache sites are recovered. Could you rebase this PR once more, @williamhyun
?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
frolovdev opened a new pull request, #1904:
URL: https://github.com/apache/arrow-rs/pull/1904
# Which issue does this PR close?
Closes #1390
--
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 t
rtpsw commented on PR #13375:
URL: https://github.com/apache/arrow/pull/13375#issuecomment-1159218051
> I skipped over the changes to nested function registries since I already
reviewed those (I think) in #13252 .
I think there are a few additions here, so I'll try to rebase to make t
rtpsw commented on PR #13252:
URL: https://github.com/apache/arrow/pull/13252#issuecomment-1159215617
This can get pushed and I'll handle the merge.
--
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 t
github-actions[bot] commented on PR #13397:
URL: https://github.com/apache/arrow/pull/13397#issuecomment-1159193573
: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 #13397:
URL: https://github.com/apache/arrow/pull/13397#issuecomment-1159193559
https://issues.apache.org/jira/browse/ARROW-16444
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
codecov-commenter commented on PR #2746:
URL:
https://github.com/apache/arrow-datafusion/pull/2746#issuecomment-1159190627
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/2746?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_
alamb commented on PR #1879:
URL: https://github.com/apache/arrow-rs/pull/1879#issuecomment-1159185058
Thanks again @DaltonModlin
--
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 com
alamb merged PR #1879:
URL: https://github.com/apache/arrow-rs/pull/1879
--
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
alamb closed issue #1843: Split up arrow::array::builder module
URL: https://github.com/apache/arrow-rs/issues/1843
--
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 unsubscri
lidavidm commented on PR #12868:
URL: https://github.com/apache/arrow/pull/12868#issuecomment-1159179680
Any more comments here? I think we can keep iterating on the documentation
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
andygrove opened a new pull request, #2746:
URL: https://github.com/apache/arrow-datafusion/pull/2746
# Which issue does this PR close?
Closes https://github.com/apache/arrow-datafusion/issues/2744
# Rationale for this change
I want to run a DISTINCT query wi
westonpace commented on code in PR #13375:
URL: https://github.com/apache/arrow/pull/13375#discussion_r900418662
##
cpp/src/arrow/engine/substrait/relation_internal.cc:
##
@@ -116,7 +119,7 @@ Result FromProto(const
substrait::Rel& rel,
} else {
return Status
andygrove opened a new pull request, #2745:
URL: https://github.com/apache/arrow-datafusion/pull/2745
# Which issue does this PR close?
N/A
# Rationale for this change
If I get an error telling me a data type is not supported then I would like
to know which
andygrove opened a new issue, #2744:
URL: https://github.com/apache/arrow-datafusion/issues/2744
**Describe the bug**
I am trying to run a DISTINCT query with a Date32 column
**To Reproduce**
Run a DISTINCT query with a Date32 column
**Expected behavior**
It should work
frolovdev commented on issue #1699:
URL: https://github.com/apache/arrow-rs/issues/1699#issuecomment-1159147393
@tustvold maybe I get your idea wrong but I can't reproduce it in any ways
```
message table {
repeated group my_group_map {
REQUIRED BYT
codecov-commenter commented on PR #1879:
URL: https://github.com/apache/arrow-rs/pull/1879#issuecomment-1159144531
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1879?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
wesm commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159136158
> An ExecBatch with all scalars does not necessarily have a length of 1.
I think we're talking about different things — many of the ScalarKernel
implementations have two implementation
alamb closed issue #1902: `dynamic_types` example does not print the projection
URL: https://github.com/apache/arrow-rs/issues/1902
--
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 commen
alamb merged PR #1903:
URL: https://github.com/apache/arrow-rs/pull/1903
--
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
alamb commented on PR #1903:
URL: https://github.com/apache/arrow-rs/pull/1903#issuecomment-1159130878
Thanks @martin-g and @paddyhoran !
--
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
westonpace commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159118738
> In the Acero execution engine, vector functions won't be supported in
expressions aside from window functions (I guess). I'm inclined to simply
disable the scalar input path here sin
westonpace commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159113163
> In [ARROW-16577](https://issues.apache.org/jira/browse/ARROW-16577) (which
I'm going to tackle within the next week hopefully), I'm going to remove the
all-scalar input path from all
wesm commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159111651
> I suppose, if the kernel functions are going to keep having scalars
without length then the correct thing to do would be to always output either a
scalar or an array of length 1.
In
wesm commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159110677
In the Acero execution engine, vector functions won't be supported in
expressions aside from window functions (I guess). I'm inclined to simply
disable the scalar input path here since it is
westonpace commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159103757
I suppose, if the kernel functions are going to keep having scalars without
length then the correct thing to do would be to always output either a scalar
or an array of length 1.
--
westonpace commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159097426
In the exec plan / engine (e.g. everywhere using ExecBatch), scalar columns
have length (part of the reason I'm a fan of treating scalars everywhere as RLE
encoded arrays is to easily
arjunsr1 opened a new issue, #13396:
URL: https://github.com/apache/arrow/issues/13396
I'm trying to use the merge function in table.rb (Line 358) and it's not
giving me the functionality I am expecting. From what I interpreted, the
function should take the table that is passed in as a para
viirya merged PR #1894:
URL: https://github.com/apache/arrow-rs/pull/1894
--
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
willbowditch commented on issue #12653:
URL: https://github.com/apache/arrow/issues/12653#issuecomment-1159030345
Finding the same thing in `pyarrow 8.0.0` converting from a CSV to Parquet -
I've tried various batch sizes on the scanner and various min/max rows/groups
on the writer.
viirya commented on code in PR #1893:
URL: https://github.com/apache/arrow-rs/pull/1893#discussion_r900294799
##
arrow/src/ipc/reader.rs:
##
@@ -702,7 +702,11 @@ pub fn read_dictionary(
DataType::Dictionary(_, ref value_type) => {
// Make a fake schema for
viirya commented on code in PR #1893:
URL: https://github.com/apache/arrow-rs/pull/1893#discussion_r900294799
##
arrow/src/ipc/reader.rs:
##
@@ -702,7 +702,11 @@ pub fn read_dictionary(
DataType::Dictionary(_, ref value_type) => {
// Make a fake schema for
wesm commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159026620
(it's also unclear to me that supporting scalar inputs to this function is
useful, but that's a separate question)
--
This is an automated message from the Apache Git Service.
To respond t
wesm commented on PR #12460:
URL: https://github.com/apache/arrow/pull/12460#issuecomment-1159026036
I'm refactoring all the vector kernels and I observed that the behavior of
this kernel is inconsistent between scalar and array inputs:
A scalar null returns an array of length with 0
westonpace commented on code in PR #13390:
URL: https://github.com/apache/arrow/pull/13390#discussion_r900283284
##
cpp/src/arrow/engine/substrait/relation_internal.cc:
##
@@ -106,25 +106,37 @@ Result FromProto(const
substrait::Rel& rel,
path = item.uri_path_glob();
westonpace commented on code in PR #13344:
URL: https://github.com/apache/arrow/pull/13344#discussion_r900281049
##
python/pyarrow/_compute.pyx:
##
@@ -2067,16 +2067,17 @@ def _group_by(args, keys, aggregations):
vector[CAggregate] c_aggregations
CDatum result
lidavidm commented on code in PR #13390:
URL: https://github.com/apache/arrow/pull/13390#discussion_r900280168
##
cpp/src/arrow/engine/substrait/relation_internal.cc:
##
@@ -106,25 +106,37 @@ Result FromProto(const
substrait::Rel& rel,
path = item.uri_path_glob();
westonpace commented on code in PR #13390:
URL: https://github.com/apache/arrow/pull/13390#discussion_r900279339
##
cpp/src/arrow/engine/substrait/relation_internal.cc:
##
@@ -106,25 +106,37 @@ Result FromProto(const
substrait::Rel& rel,
path = item.uri_path_glob();
marsupialtail commented on code in PR #13385:
URL: https://github.com/apache/arrow/pull/13385#discussion_r900266915
##
cpp/src/arrow/filesystem/s3fs.h:
##
@@ -102,6 +102,8 @@ struct ARROW_EXPORT S3Options {
/// the region (environment variables, configuration profile, EC2 met
drin commented on PR #13383:
URL: https://github.com/apache/arrow/pull/13383#issuecomment-1158969053
yep, I think so!
--
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 uns
codecov-commenter commented on PR #1903:
URL: https://github.com/apache/arrow-rs/pull/1903#issuecomment-1158951662
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1903?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
zagto commented on code in PR #1:
URL: https://github.com/apache/arrow/pull/1#discussion_r900191747
##
docs/source/format/Columnar.rst:
##
@@ -765,6 +765,65 @@ application.
We discuss dictionary encoding as it relates to serialization further
below.
+.. _run-length-e
andygrove opened a new issue, #2743:
URL: https://github.com/apache/arrow-datafusion/issues/2743
**Describe the bug**
The discord link in
https://arrow.apache.org/datafusion/community/communication.html#slack-and-discord
has expired
Perhaps we should just ask people to email dev@
andygrove commented on issue #2709:
URL:
https://github.com/apache/arrow-datafusion/issues/2709#issuecomment-1158922350
> The main issue a few months ago was that the datafusion codebase was not
split like it is now, so it created conflicts in the various places where it
wasn't possible to
andygrove commented on issue #2709:
URL:
https://github.com/apache/arrow-datafusion/issues/2709#issuecomment-1158912362
> I do think there is broad agreement that the governance model of arrow2
(benign dictator) is not suitable for all users for a variety of reasons.
I agree. Althoug
martin-g opened a new pull request, #1903:
URL: https://github.com/apache/arrow-rs/pull/1903
# Which issue does this PR close?
Closes #1902.
# Rationale for this change
The user can get a feeling of the data.
# What changes are included in this PR?
martin-g opened a new issue, #1902:
URL: https://github.com/apache/arrow-rs/issues/1902
**Describe the bug**
https://github.com/apache/arrow-rs/blob/master/arrow/examples/dynamic_types.rs
shows how to use and project `RecordBatch`es.
I think it would be a slightly more useful if i
lidavidm commented on PR #13383:
URL: https://github.com/apache/arrow/pull/13383#issuecomment-1158893351
Looks like everything's passing. Is this ready @drin?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
martin-g closed pull request #1895: Issue #1876: Explicitly declare the used
features for each dependency in parquet
URL: https://github.com/apache/arrow-rs/pull/1895
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
martin-g closed pull request #1897: Issue #1876: Explicitly declare the used
features for each dependency in parquet_derive_test
URL: https://github.com/apache/arrow-rs/pull/1897
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
HaoYang670 commented on issue #1901:
URL: https://github.com/apache/arrow-rs/issues/1901#issuecomment-1158847977
Ha! Maybe we could remove this function directly because we already have
`num_required_bits` which provide same functionality.
https://github.com/apache/arrow-rs/blob/master/
HaoYang670 commented on issue #1901:
URL: https://github.com/apache/arrow-rs/issues/1901#issuecomment-1158844111
Thank you @jhorstmann.
Agree with you on using `leading_zeros`, and this is also how the std
library implements:
```rust
#[inline]
pub
codecov-commenter commented on PR #1900:
URL: https://github.com/apache/arrow-rs/pull/1900#issuecomment-1158841833
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1900?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
jhorstmann commented on issue #1901:
URL: https://github.com/apache/arrow-rs/issues/1901#issuecomment-1158840184
Since this seems to be only used for getting the width of bitpacked data, a
faster way to calculate that is using the `leading_zeros` function. I
contributed a similar improvemen
cyb70289 commented on PR #13394:
URL: https://github.com/apache/arrow/pull/13394#issuecomment-1158838873
From conbench, csv writer benchmark improvement on `i9` is about 8% to 25%.
`m1` improvement is small (5% to 10%).
--
This is an automated message from the Apache Git Service.
To respo
github-actions[bot] commented on PR #13395:
URL: https://github.com/apache/arrow/pull/13395#issuecomment-1158835036
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
HaoYang670 opened a new issue, #1901:
URL: https://github.com/apache/arrow-rs/issues/1901
**Describe the bug**
https://github.com/apache/arrow-rs/blob/master/parquet/src/util/bit_util.rs#L142-L155
**To Reproduce**
assert_eq!(log2(0), 0);
**Expected behavior**
Alter 1:
jhorstmann opened a new pull request, #1900:
URL: https://github.com/apache/arrow-rs/pull/1900
# Which issue does this PR close?
Closes #1899.
# Rationale for this change
The buffers are storing validity bits and there the offset has to be
interpreted as number o
lidavidm commented on code in PR #13109:
URL: https://github.com/apache/arrow/pull/13109#discussion_r90006
##
python/pyarrow/tests/test_compute.py:
##
@@ -1702,13 +1702,37 @@ def test_logical():
def test_cast():
+arr = pa.array([1, 2, 3, 4], type='int64')
+optio
assignUser closed pull request #13240: ARROW-16406: [Docs][R] Update
documentation with new nightly location
URL: https://github.com/apache/arrow/pull/13240
--
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
assignUser commented on PR #13240:
URL: https://github.com/apache/arrow/pull/13240#issuecomment-1158810321
Task merged with #13241/ARROW-16405
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
lidavidm commented on issue #13391:
URL: https://github.com/apache/arrow/issues/13391#issuecomment-1158809150
They're tested by the integration suite, e.g. see the output of this run:
https://github.com/apache/arrow/runs/6927219399?check_suite_focus=true#step:7:11104
--
This is an automat
codecov-commenter commented on PR #1898:
URL: https://github.com/apache/arrow-rs/pull/1898#issuecomment-1158807754
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1898?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
jhorstmann opened a new issue, #1899:
URL: https://github.com/apache/arrow-rs/issues/1899
**Describe the bug**
In
[`combine_option_bitmap`](https://github.com/apache/arrow-rs/blob/fc4044f35d4aa67e706c6d3f61a9f24bab5346be/arrow/src/compute/util.rs#L58)
the buffer is sliced using `Buff
frolovdev commented on issue #1642:
URL: https://github.com/apache/arrow-rs/issues/1642#issuecomment-1158804062
@tustvold
So the basic idea is to avoid the obligation of values in the map. According
to
```
The value field encodes the map's value type and repetition. This field
lidavidm commented on code in PR #13390:
URL: https://github.com/apache/arrow/pull/13390#discussion_r900050308
##
cpp/src/arrow/util/uri.h:
##
@@ -68,6 +68,8 @@ class ARROW_EXPORT Uri {
/// The URI path component.
std::string path() const;
+ std::string extension() cons
martin-g opened a new pull request, #1898:
URL: https://github.com/apache/arrow-rs/pull/1898
# Which issue does this PR close?
Closes #1876.
This is the last PR for https://github.com/apache/arrow-rs/issues/1876.
It changes just integration_testing/Cargo.toml.
The PR does **no
codecov-commenter commented on PR #1897:
URL: https://github.com/apache/arrow-rs/pull/1897#issuecomment-1158790046
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/1897?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+S
martin-g opened a new pull request, #1897:
URL: https://github.com/apache/arrow-rs/pull/1897
# Which issue does this PR close?
This is a PR for https://github.com/apache/arrow-rs/issues/1876.
It changes just parquet_derive_test/Cargo.toml.
The PR does not upgrade the dependencies
martin-g opened a new pull request, #1896:
URL: https://github.com/apache/arrow-rs/pull/1896
# Which issue does this PR close?
This is a PR for https://github.com/apache/arrow-rs/issues/1876.
It changes just parquet_derive/Cargo.toml.
The PR does **not** upgrade the dependencies!
martin-g opened a new pull request, #1895:
URL: https://github.com/apache/arrow-rs/pull/1895
Declare that parquet module uses rand's std and std_rng features
# Which issue does this PR close?
This is the third PR for https://github.com/apache/arrow-rs/issues/1876.
It changes
alamb commented on code in PR #1891:
URL: https://github.com/apache/arrow-rs/pull/1891#discussion_r99399
##
arrow/src/compute/kernels/temporal.rs:
##
@@ -211,6 +211,34 @@ where
Ok(b.finish())
}
+/// Extracts the day of week of a given temporal array as an array of in
alamb opened a new pull request, #1894:
URL: https://github.com/apache/arrow-rs/pull/1894
Minor follow on to https://github.com/apache/arrow-rs/pull/1891 from @nl5887
-- add an example in the docstring of `weekday` kernel
--
This is an automated message from the Apache Git Service.
To res
alamb merged PR #1891:
URL: https://github.com/apache/arrow-rs/pull/1891
--
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
alamb commented on code in PR #1891:
URL: https://github.com/apache/arrow-rs/pull/1891#discussion_r96406
##
arrow/src/compute/kernels/temporal.rs:
##
@@ -211,6 +211,34 @@ where
Ok(b.finish())
}
+/// Extracts the day of week of a given temporal array as an array of in
1 - 100 of 125 matches
Mail list logo
