dependabot[bot] opened a new pull request, #253:
URL: https://github.com/apache/arrow-dotnet/pull/253
Updated [Google.Protobuf](https://github.com/protocolbuffers/protobuf) from
3.31.1 to 3.33.5.
Release notes
_Sourced from [Google.Protobuf's
releases](https://github.com/p
mobiusklein commented on PR #251:
URL: https://github.com/apache/arrow-dotnet/pull/251#issuecomment-3832832418
Thank you. I misunderstood the argument being made, not that the underlying
hardware was not 8-bit byte compliant but that the library defined types did
not enforce that invariant.
mobiusklein opened a new pull request, #251:
URL: https://github.com/apache/arrow-dotnet/pull/251
## What's Changed
Uses unsigned 64-bit integers to compute the intermediate bit length of an
array to avoid arithmetic overflow.
I could factorize this expression differently to a
kevinjqliu opened a new pull request, #631:
URL: https://github.com/apache/arrow-rs-object-store/pull/631
# Which issue does this PR close?
Closes #.
# Rationale for this change
Fixes a typo in the Azure URL parser where `blob.fabric.microsoft.net` was
used
kevinjqliu commented on PR #631:
URL:
https://github.com/apache/arrow-rs-object-store/pull/631#issuecomment-3832336898
noticed this while reviewing the fabric code.
`windows.net` is valid, fabric uses `fabric.microsoft.com`
--
This is an automated message from the Apache Git S
qiushido commented on issue #622:
URL: https://github.com/apache/arrow-go/issues/622#issuecomment-3832394767
@zeroshade Thank you for finding the problem and deciding to fix it. I think
your analysis is the root cause of the problem, because my panic usually occurs
when a certain field is r
github-actions[bot] commented on PR #49111:
URL: https://github.com/apache/arrow/pull/49111#issuecomment-3832411450
:warning: GitHub issue #49110 **has been automatically assigned in GitHub**
to PR creator.
--
This is an automated message from the Apache Git Service.
To respond to the mes
adamreeve commented on code in PR #251:
URL: https://github.com/apache/arrow-dotnet/pull/251#discussion_r2752192644
##
src/Apache.Arrow/C/CArrowArrayImporter.cs:
##
@@ -470,7 +470,7 @@ private ArrowBuffer[] ImportFixedWidthBuffers(CArrowArray*
cArray, int bitWidth)
HyukjinKwon opened a new pull request, #49111:
URL: https://github.com/apache/arrow/pull/49111
### Rationale for this change
Previously, `DataType::field(int i)` directly accessed `children_[i]`
without bounds checking, resulting in undefined behavior for out-of-bounds
indices. This
github-actions[bot] commented on PR #995:
URL: https://github.com/apache/arrow-java/pull/995#issuecomment-3832548034
Thank you for opening a pull request!
Please label the PR with one or more of:
- bug-fix
- chore
- dependencies
- documentation
- enhancement
ennuite opened a new pull request, #995:
URL: https://github.com/apache/arrow-java/pull/995
## What's Changed
Multiple DatabaseMetaData methods had NPEs when the method
`ArrowDatabaseMetadata.getSqlInfoAndCacheIfCacheIsEmpty(final SqlInfo
sqlInfoCommand, final Class desiredType)`
HyukjinKwon commented on code in PR #49113:
URL: https://github.com/apache/arrow/pull/49113#discussion_r2752295603
##
cpp/src/arrow/tensor_test.cc:
##
@@ -505,8 +506,16 @@ TYPED_TEST_P(TestFloatTensor, Equals) {
std::vector shape = {4, 4};
- std::vector c_values = {1, 2,
adamreeve commented on code in PR #251:
URL: https://github.com/apache/arrow-dotnet/pull/251#discussion_r2752376460
##
src/Apache.Arrow/C/CArrowArrayImporter.cs:
##
@@ -470,7 +470,7 @@ private ArrowBuffer[] ImportFixedWidthBuffers(CArrowArray*
cArray, int bitWidth)
dependabot[bot] opened a new pull request, #252:
URL: https://github.com/apache/arrow-dotnet/pull/252
Bumps [actions/cache](https://github.com/actions/cache) from 5.0.2 to 5.0.3.
Release notes
Sourced from https://github.com/actions/cache/releases";>actions/cache's
releases.
ennuite commented on issue #994:
URL: https://github.com/apache/arrow-java/issues/994#issuecomment-3832596023
@merzbird created a PR. Let me know if you agree with the fix, I return
sensible values like you suggest, and for classes for which no known decent
defaults are know, I throw a SQLE
mobiusklein commented on code in PR #251:
URL: https://github.com/apache/arrow-dotnet/pull/251#discussion_r2752257564
##
src/Apache.Arrow/C/CArrowArrayImporter.cs:
##
@@ -470,7 +470,7 @@ private ArrowBuffer[] ImportFixedWidthBuffers(CArrowArray*
cArray, int bitWidth)
HyukjinKwon opened a new pull request, #49113:
URL: https://github.com/apache/arrow/pull/49113
### Rationale for this change
Half-float (float16) tensor equality comparison was not implemented. The
TODO comment indicated this was missing functionality. Array/Scalar equality
for half-
Divyanshu-s13 commented on PR #344:
URL: https://github.com/apache/arrow-js/pull/344#issuecomment-3832862896
@kou I fixed this issue. Could you please merge it?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
DivineDominion commented on PR #125:
URL: https://github.com/apache/arrow-swift/pull/125#issuecomment-3832867517
Tagged releases are indexed and documentation is generated 'immediately' but
for main branch updates, they run nightly jobs to check only, so it may take a
while. @kou
--
Thi
Divyanshu-s13 commented on issue #49:
URL: https://github.com/apache/arrow-js/issues/49#issuecomment-3832869184
@jackvial @kou @raulcd could you please assign this issue to me? I’m
currently working on this repository and can take care of the fix.
--
This is an automated message from th
viky-01 commented on issue #46531:
URL: https://github.com/apache/arrow/issues/46531#issuecomment-3833118185
@vipin1904 yes, it is done
--
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 specif
emkornfield commented on code in PR #48635:
URL: https://github.com/apache/arrow/pull/48635#discussion_r2751995905
##
cpp/src/parquet/row_selection.h:
##
@@ -0,0 +1,79 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
emkornfield commented on code in PR #48635:
URL: https://github.com/apache/arrow/pull/48635#discussion_r2751994715
##
cpp/src/parquet/row_selection.h:
##
@@ -0,0 +1,79 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
emkornfield commented on code in PR #48635:
URL: https://github.com/apache/arrow/pull/48635#discussion_r2751997485
##
cpp/src/parquet/row_selection.h:
##
@@ -0,0 +1,79 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
emkornfield commented on code in PR #48635:
URL: https://github.com/apache/arrow/pull/48635#discussion_r2751996953
##
cpp/src/parquet/row_selection.h:
##
@@ -0,0 +1,79 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
emkornfield commented on code in PR #48635:
URL: https://github.com/apache/arrow/pull/48635#discussion_r2751996953
##
cpp/src/parquet/row_selection.h:
##
@@ -0,0 +1,79 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
emkornfield commented on PR #48635:
URL: https://github.com/apache/arrow/pull/48635#issuecomment-3832110989
Took a look at the top-level public API, looks reasonable to me. If you
need a full review please ping me.
Apologies for the delay.
--
This is an automated message from the
adriangb commented on code in PR #9138:
URL: https://github.com/apache/arrow-rs/pull/9138#discussion_r2751328890
##
arrow-array/src/heap_size.rs:
##
@@ -0,0 +1,138 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See
adriangb commented on code in PR #9138:
URL: https://github.com/apache/arrow-rs/pull/9138#discussion_r2751328438
##
arrow-memory-size/README.md:
##
@@ -0,0 +1,231 @@
+
+
+# `arrow-memory-size`
+
+[](https://crate
eddelbuettel commented on PR #842:
URL: https://github.com/apache/arrow-nanoarrow/pull/842#issuecomment-3831161312
Thanks for shepherding it to CRAN!
One thing that is a little ... weird is that I cannot find a list of changes
defining 0.7.0-3-1 anywhere. Not here, not in releases,
adriangb commented on code in PR #9138:
URL: https://github.com/apache/arrow-rs/pull/9138#discussion_r2751327162
##
arrow-memory-size/src/lib.rs:
##
@@ -0,0 +1,688 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See
kou commented on PR #49105:
URL: https://github.com/apache/arrow/pull/49105#issuecomment-3830992211
Could you add a test for this case?
--
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
shinmao opened a new issue, #9324:
URL: https://github.com/apache/arrow-rs/issues/9324
The function `BitReader::get_batch` in the `experimental` feature assumes
types with `size_of::() == N` have alignment `>= N`, but this is not
enforced. Users implementing `FromBytes` for `#[repr(packed)]
jhorstmann opened a new pull request, #9325:
URL: https://github.com/apache/arrow-rs/pull/9325
# Which issue does this PR close?
- Closes #9323.
# Rationale for this change
This bug seems to have existed for at least the last 5 years (before that,
array equality
HyukjinKwon commented on issue #49112:
URL: https://github.com/apache/arrow/issues/49112#issuecomment-3832489305
I am taking a look at this.
--
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
AliRana30 commented on PR #49086:
URL: https://github.com/apache/arrow/pull/49086#issuecomment-3831214230
Hi @kou,
Thank you for the feedback!
**### Regarding Table::Make and DCHECK:** You are absolutely right. I have
refactored Table::Make (and Table::MakeEmpty) to return
Res
AliRana30 commented on PR #49105:
URL: https://github.com/apache/arrow/pull/49105#issuecomment-3831326949
@kou On your request, I have added a new test case,
`TestEqualityMismatchedDimensions`, in
`cpp/src/arrow/sparse_tensor_test.cc`.
**Test details:**
This test compares `S
alchemist51 commented on code in PR #9138:
URL: https://github.com/apache/arrow-rs/pull/9138#discussion_r2751559818
##
arrow-memory-size/README.md:
##
@@ -0,0 +1,231 @@
+
+
+# `arrow-memory-size`
+
+[](https://cr
AliRana30 commented on PR #49086:
URL: https://github.com/apache/arrow/pull/49086#issuecomment-3831279779
Hi @WillAyd,
I've added the requested tests in `cpp/src/arrow/table_test.cc` to verify
the fix.
**Specifically, I added:**
- `TestTable.MakeInvalidInputs`: Verifies that
EmilyMatt commented on PR #8930:
URL: https://github.com/apache/arrow-rs/pull/8930#issuecomment-3831816665
Are there any further blockers? This is ready on my end.
@jecsand838 @alamb @mzabaluev
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
jhorstmann commented on issue #9323:
URL: https://github.com/apache/arrow-rs/issues/9323#issuecomment-3831787141
Wow, good find. This bug seems to have existed for at least the last 5
years. I'll look into a fix.
--
This is an automated message from the Apache Git Service.
To respond to t
Divyanshu-s13 commented on PR #339:
URL: https://github.com/apache/arrow-js/pull/339#issuecomment-3833480557
@kou Thanks for the review and for merging this! 🙌
I enjoyed working on this and will continue exploring the codebase to
contribute more.
--
This is an automated message
Divyanshu-s13 commented on PR #344:
URL: https://github.com/apache/arrow-js/pull/344#issuecomment-3833485544
@kou By the way, I’m planning to apply for GSoC this year and really
enjoying contributing here.
I’d love to align my work with areas that matter most to the project.
--
This
shinmao commented on issue #9324:
URL: https://github.com/apache/arrow-rs/issues/9324#issuecomment-3833502016
@Jefffrey thanks for the response. I think you solution is correct to add
requirement on unsafe trait. It will bridge the gap between unsafe trait and
unsafe slice function called i
Divyanshu-s13 commented on PR #341:
URL: https://github.com/apache/arrow-js/pull/341#issuecomment-3833503916
@kou @domoritz please review it.
--
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
dependabot[bot] opened a new pull request, #996:
URL: https://github.com/apache/arrow-java/pull/996
Bumps [docker/login-action](https://github.com/docker/login-action) from
3.6.0 to 3.7.0.
Release notes
Sourced from https://github.com/docker/login-action/releases";>docker/login-act
dependabot[bot] opened a new pull request, #1000:
URL: https://github.com/apache/arrow-java/pull/1000
Bumps [org.mockito:mockito-bom](https://github.com/mockito/mockito) from
5.17.0 to 5.21.0.
Release notes
Sourced from https://github.com/mockito/mockito/releases";>org.mockito:mock
dependabot[bot] opened a new pull request, #998:
URL: https://github.com/apache/arrow-java/pull/998
Bumps
[com.gradle:common-custom-user-data-maven-extension](https://github.com/gradle/common-custom-user-data-maven-extension)
from 2.0.3 to 2.1.0.
Release notes
Sourced from https:/
dependabot[bot] opened a new pull request, #997:
URL: https://github.com/apache/arrow-java/pull/997
Bumps [commons-codec:commons-codec](https://github.com/apache/commons-codec)
from 1.20.0 to 1.21.0.
Changelog
Sourced from https://github.com/apache/commons-codec/blob/master/RELEASE
dependabot[bot] opened a new pull request, #999:
URL: https://github.com/apache/arrow-java/pull/999
Bumps `logback.version` from 1.5.26 to 1.5.27.
Updates `ch.qos.logback:logback-classic` from 1.5.26 to 1.5.27
Release notes
Sourced from https://github.com/qos-ch/logback/releases"
dependabot[bot] opened a new pull request, #1001:
URL: https://github.com/apache/arrow-java/pull/1001
Bumps com.gradle:develocity-maven-extension from 2.3.1 to 2.3.3.
[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-n
Jefffrey commented on issue #9324:
URL: https://github.com/apache/arrow-rs/issues/9324#issuecomment-3833357976
Would the fix here be as simple as fixing the documentation for `FromBytes`
to include alignment requirements?
https://github.com/apache/arrow-rs/blob/860b2db748f11fc9396079
albertlockett opened a new pull request, #9322:
URL: https://github.com/apache/arrow-rs/pull/9322
# Which issue does this PR close?
- Closes #9321 .
# Rationale for this change
Improve the performance of creating a dictionary array that is all nulls.
albertlockett commented on PR #9322:
URL: https://github.com/apache/arrow-rs/pull/9322#issuecomment-3831084865
Benchmark results:
Before:
```
$ cargo bench --bench null_dict -- "null_dict"
null_dict/len=128 time: [132.45 ns 133.37 ns 134.59 ns]
null_dict/len
albertlockett commented on PR #9322:
URL: https://github.com/apache/arrow-rs/pull/9322#issuecomment-3831086584
Here is a profile I captured from the benchmark execution before these
changes were made. We see much time spent validating the dictionary keys:
https://github.com/user-attac
pierrebelzile opened a new issue, #9323:
URL: https://github.com/apache/arrow-rs/issues/9323
**Describe the bug**
Two arrays compare equal if the characters are the same but with different
offset lengths.
**To Reproduce**
```
use arrow::array::Array;
#[test]
fn
albertlockett opened a new issue, #9321:
URL: https://github.com/apache/arrow-rs/issues/9321
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
We have a `RecordBatch` in our application that is has this form where
there's va
chilin0525 opened a new pull request, #49109:
URL: https://github.com/apache/arrow/pull/49109
### Rationale for this change
`SparseCOOTensor.__repr__` outputs literal `{self.type}` and `{self.shape}`
instead of actual values due to missing f-string prefix.
### What changes are
github-actions[bot] commented on PR #49109:
URL: https://github.com/apache/arrow/pull/49109#issuecomment-3830616994
:warning: GitHub issue #49108 **has been automatically assigned in GitHub**
to PR creator.
--
This is an automated message from the Apache Git Service.
To respond to the mes
AliRana30 commented on issue #49104:
URL: https://github.com/apache/arrow/issues/49104#issuecomment-3830664154
@kou Hi Kou! I've identified and fixed a critical segfault in
`SparseCSFIndex::Equals` that occurs during mismatched dimension comparisons.
Could you please take a look at this
AliRana30 commented on PR #49086:
URL: https://github.com/apache/arrow/pull/49086#issuecomment-3830702896
Hi @kou I have updated the PR to follow your suggestion. I've moved the
validation to Table::Make using DCHECK to ensure that callers provide valid
(non-null) arrays. I also reverted th
github-actions[bot] commented on issue #28876:
URL: https://github.com/apache/arrow/issues/28876#issuecomment-3830901903
This issue has been marked as stale because it has had no activity in the
past 365 days. Please remove the stale label or comment below, or this issue
will be closed in 1
github-actions[bot] commented on issue #28887:
URL: https://github.com/apache/arrow/issues/28887#issuecomment-3830901953
This issue has been marked as stale because it has had no activity in the
past 365 days. Please remove the stale label or comment below, or this issue
will be closed in 1
github-actions[bot] commented on issue #28899:
URL: https://github.com/apache/arrow/issues/28899#issuecomment-3830901971
This issue has been marked as stale because it has had no activity in the
past 365 days. Please remove the stale label or comment below, or this issue
will be closed in 1
github-actions[bot] commented on issue #18725:
URL: https://github.com/apache/arrow/issues/18725#issuecomment-3830901994
This issue has been marked as stale because it has had no activity in the
past 365 days. Please remove the stale label or comment below, or this issue
will be closed in 1
github-actions[bot] commented on issue #28932:
URL: https://github.com/apache/arrow/issues/28932#issuecomment-3830902046
This issue has been marked as stale because it has had no activity in the
past 365 days. Please remove the stale label or comment below, or this issue
will be closed in 1
github-actions[bot] commented on issue #18729:
URL: https://github.com/apache/arrow/issues/18729#issuecomment-3830902015
This issue has been marked as stale because it has had no activity in the
past 365 days. Please remove the stale label or comment below, or this issue
will be closed in 1
github-actions[bot] commented on issue #28942:
URL: https://github.com/apache/arrow/issues/28942#issuecomment-3830902076
This issue has been marked as stale because it has had no activity in the
past 365 days. Please remove the stale label or comment below, or this issue
will be closed in 1
kou commented on PR #49086:
URL: https://github.com/apache/arrow/pull/49086#issuecomment-3830990447
Why did you use `DCHECK` instead of `arrow::Status`? We can use
`arrow::Status` by changing return type of `Table::Make` to
`Result>` from `std::shared_ptr`. `DCHECK` is
evaluated only with
69 matches
Mail list logo
