mapleFU commented on code in PR #45421:
URL: https://github.com/apache/arrow/pull/45421#discussion_r1942372346
##
cpp/src/arrow/acero/concurrent_queue_internal.h:
##
@@ -35,8 +35,7 @@ class ConcurrentQueue {
//
T Pop() {
std::unique_lock lock(mutex_);
-cond_.wait(
gitmodimo commented on code in PR #45421:
URL: https://github.com/apache/arrow/pull/45421#discussion_r1942369015
##
cpp/src/arrow/acero/concurrent_queue_internal.h:
##
@@ -35,8 +35,7 @@ class ConcurrentQueue {
//
T Pop() {
std::unique_lock lock(mutex_);
-cond_.wai
kou commented on issue #574:
URL: https://github.com/apache/arrow-java/issues/574#issuecomment-2635842172
I'll create RC2 tomorrow.
--
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 co
singh1203 commented on PR #250:
URL: https://github.com/apache/arrow-go/pull/250#issuecomment-2635809993
Hi @kou Maybe we could do another runner's round.
--
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
kou commented on issue #587:
URL: https://github.com/apache/arrow-java/issues/587#issuecomment-2635838722
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 specific comment.
To unsubsc
kou opened a new pull request, #589:
URL: https://github.com/apache/arrow-java/pull/589
Fixes GH-588.
--
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
kou commented on issue #574:
URL: https://github.com/apache/arrow-java/issues/574#issuecomment-2635830474
Hmm...
It reached a secondary rate limit again...
https://github.com/apache/arrow-java/actions/runs/13151205894/job/36700236201#step:3:17
```text
HTTP 403: You ha
jbonofre commented on issue #587:
URL: https://github.com/apache/arrow-java/issues/587#issuecomment-2635822953
Yup, good point. Let me propose a PR about that if it works for you 😄
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
kou commented on issue #267:
URL: https://github.com/apache/arrow-go/issues/267#issuecomment-2635812268
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 specific comment.
To unsubscri
singh1203 commented on issue #267:
URL: https://github.com/apache/arrow-go/issues/267#issuecomment-2635806594
Hey @kou I would be happy to assist here; kindly assign
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
kou commented on PR #578:
URL: https://github.com/apache/arrow-java/pull/578#issuecomment-2635768950
Thanks! I'll create cut an RC.
--
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 co
mapleFU commented on code in PR #45370:
URL: https://github.com/apache/arrow/pull/45370#discussion_r1942275704
##
cpp/src/arrow/scalar_test.cc:
##
@@ -156,6 +157,45 @@ TEST(TestBooleanScalar, Cast) {
}
}
+TEST(TestScalar, IndentityCast) {
+ random::RandomArrayGenerator ge
mapleFU commented on code in PR #45370:
URL: https://github.com/apache/arrow/pull/45370#discussion_r1942275285
##
cpp/src/arrow/scalar_test.cc:
##
@@ -866,6 +866,9 @@ TEST(TestTimeScalars, Basics) {
ASSERT_TRUE(first->Equals(*MakeScalar(ty, 5).ValueOrDie()));
ASSERT_TR
jbonofre commented on PR #578:
URL: https://github.com/apache/arrow-java/pull/578#issuecomment-2635744222
@lidavidm @kou thanks for the review and merge. I'm doing a quick new run
but I think we are good for the release.
--
This is an automated message from the Apache Git Service.
To res
lidavidm merged PR #578:
URL: https://github.com/apache/arrow-java/pull/578
--
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
jbonofre commented on PR #578:
URL: https://github.com/apache/arrow-java/pull/578#issuecomment-2635736474
> @jbonofre Can we merge this?
Yes, we are good 👍
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
irenjj commented on code in PR #7065:
URL: https://github.com/apache/arrow-rs/pull/7065#discussion_r1942246993
##
arrow-cast/src/cast/list.rs:
##
@@ -88,11 +88,16 @@ where
let mut mutable = MutableArrayData::new(vec![&values], nullable, cap);
// The end position in val
amoeba commented on PR #45425:
URL: https://github.com/apache/arrow/pull/45425#issuecomment-2635688308
Thanks for getting the jobs running, I'll check on them tomorrow.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
amoeba commented on code in PR #45425:
URL: https://github.com/apache/arrow/pull/45425#discussion_r1942223874
##
ci/scripts/python_wheel_windows_test.bat:
##
@@ -58,11 +58,14 @@ py -0p
@REM Validate wheel contents
%PYTHON_CMD% C:\arrow\ci\scripts\python_wheel_validate_contents
github-actions[bot] commented on PR #45425:
URL: https://github.com/apache/arrow/pull/45425#issuecomment-2635664530
Revision: 29467e206d85e5101743a8783ef9fc5b8e210781
Submitted crossbow builds: [ursacomputing/crossbow @
actions-435df0c2a2](https://github.com/ursacomputing/crossbow/bra
amoeba commented on code in PR #45425:
URL: https://github.com/apache/arrow/pull/45425#discussion_r1942221651
##
ci/scripts/python_wheel_windows_test.bat:
##
@@ -58,11 +58,14 @@ py -0p
@REM Validate wheel contents
%PYTHON_CMD% C:\arrow\ci\scripts\python_wheel_validate_contents
kou commented on PR #45425:
URL: https://github.com/apache/arrow/pull/45425#issuecomment-2635662397
@github-actions crossbow submit wheel-windows-*
--
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
github-actions[bot] commented on PR #45425:
URL: https://github.com/apache/arrow/pull/45425#issuecomment-2635655973
```
Unable to match any tasks for `wheel-windows-cp39-amd64`
The Archery job run can be found at:
https://github.com/apache/arrow/actions/runs/13149993032
```
--
Th
kou commented on PR #45425:
URL: https://github.com/apache/arrow/pull/45425#issuecomment-2635654747
@github-actions crossbow submit wheel-windows-cp39-amd64
--
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
kou commented on code in PR #45425:
URL: https://github.com/apache/arrow/pull/45425#discussion_r1942215856
##
ci/scripts/python_wheel_windows_test.bat:
##
@@ -58,11 +58,14 @@ py -0p
@REM Validate wheel contents
%PYTHON_CMD% C:\arrow\ci\scripts\python_wheel_validate_contents.py
kou commented on code in PR #45425:
URL: https://github.com/apache/arrow/pull/45425#discussion_r1942215482
##
ci/scripts/python_wheel_windows_test.bat:
##
@@ -58,11 +58,14 @@ py -0p
@REM Validate wheel contents
%PYTHON_CMD% C:\arrow\ci\scripts\python_wheel_validate_contents.py
kou commented on code in PR #45425:
URL: https://github.com/apache/arrow/pull/45425#discussion_r1942213246
##
ci/scripts/python_wheel_windows_test.bat:
##
@@ -58,11 +58,14 @@ py -0p
@REM Validate wheel contents
%PYTHON_CMD% C:\arrow\ci\scripts\python_wheel_validate_contents.py
amoeba commented on PR #45425:
URL: https://github.com/apache/arrow/pull/45425#issuecomment-2635637641
@kou is there a way to test this PR?
--
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
amoeba commented on code in PR #45425:
URL: https://github.com/apache/arrow/pull/45425#discussion_r1942203941
##
ci/scripts/python_wheel_windows_test.bat:
##
@@ -58,11 +58,14 @@ py -0p
@REM Validate wheel contents
%PYTHON_CMD% C:\arrow\ci\scripts\python_wheel_validate_contents
amoeba commented on code in PR #45425:
URL: https://github.com/apache/arrow/pull/45425#discussion_r1942205440
##
ci/scripts/python_wheel_windows_test.bat:
##
@@ -58,11 +58,14 @@ py -0p
@REM Validate wheel contents
%PYTHON_CMD% C:\arrow\ci\scripts\python_wheel_validate_contents
amoeba commented on code in PR #45425:
URL: https://github.com/apache/arrow/pull/45425#discussion_r1942203941
##
ci/scripts/python_wheel_windows_test.bat:
##
@@ -58,11 +58,14 @@ py -0p
@REM Validate wheel contents
%PYTHON_CMD% C:\arrow\ci\scripts\python_wheel_validate_contents
amoeba commented on issue #45295:
URL: https://github.com/apache/arrow/issues/45295#issuecomment-2635632790
After reopening this and putting up a PR, I realize the issue here is only
related but not the same as what's being fixed in
https://github.com/apache/arrow/pull/45425. Should I open
github-actions[bot] commented on PR #45425:
URL: https://github.com/apache/arrow/pull/45425#issuecomment-2635627294
```
Unable to match any tasks for `wheel-windows-cp39-amd64`
The Archery job run can be found at:
https://github.com/apache/arrow/actions/runs/13149682761
```
--
Th
amoeba commented on PR #45425:
URL: https://github.com/apache/arrow/pull/45425#issuecomment-2635626316
@github-actions crossbow submit wheel-windows-cp39-amd64
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
amoeba commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635624315
PR over at https://github.com/apache/arrow/pull/45425. I didn't test the
whole script since it needs to be tested as part of CI.
--
This is an automated message from the Apache Git Servi
amoeba opened a new pull request, #45425:
URL: https://github.com/apache/arrow/pull/45425
### Rationale for this change
The current version of this script pulls the tzinfo database from a URL
that's not stable. Instead of finding another stable URL or hosting a version
ourselves, @ko
github-actions[bot] commented on PR #45425:
URL: https://github.com/apache/arrow/pull/45425#issuecomment-2635623554
:warning: GitHub issue #45295 **has been automatically assigned in GitHub**
to PR creator.
--
This is an automated message from the Apache Git Service.
To respond to the mes
amoeba commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635606090
Will do that in a sec once I figure out the .bat script part.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
kou commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635604083
Wow! Could you open a PR that uses https://pypi.org/project/tzdata/ ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
amoeba commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635598251
That seems like it could work,
```sh
❯ python -c "import os; from importlib import resources;
print(os.path.join(resources.files('tzdata'), 'zoneinfo'));"
C:\Users\Bryce\Temp\t
kou commented on issue #44761:
URL: https://github.com/apache/arrow/issues/44761#issuecomment-2635597600
Issue resolved by pull request 45414
https://github.com/apache/arrow/pull/45414
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
kou merged PR #45414:
URL: https://github.com/apache/arrow/pull/45414
--
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
wgtmac commented on code in PR #45424:
URL: https://github.com/apache/arrow/pull/45424#discussion_r1942182890
##
cpp/cmake_modules/ThirdpartyToolchain.cmake:
##
@@ -1256,14 +1256,19 @@ endif()
# - Gandiva has a compile-time (header-only) dependency on Boost, not runtime.
# - T
kou commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635586226
Hmm... I think that we don't want to update tzdata periodically...
We may be able to use https://pypi.org/project/tzdata/ .
--
This is an automated message from the Apache Git Service
mapleFU commented on code in PR #45421:
URL: https://github.com/apache/arrow/pull/45421#discussion_r1942176129
##
cpp/src/arrow/acero/concurrent_queue_internal.h:
##
@@ -35,8 +35,7 @@ class ConcurrentQueue {
//
T Pop() {
std::unique_lock lock(mutex_);
-cond_.wait(
amoeba commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635583424
Thanks @kou. Does it make sense for us to host a copy ourselves?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
kou commented on code in PR #45424:
URL: https://github.com/apache/arrow/pull/45424#discussion_r1942166829
##
cpp/cmake_modules/ThirdpartyToolchain.cmake:
##
@@ -1256,14 +1256,19 @@ endif()
# - Gandiva has a compile-time (header-only) dependency on Boost, not runtime.
# - Test
kou commented on PR #578:
URL: https://github.com/apache/arrow-java/pull/578#issuecomment-2635569182
@jbonofre Can we merge 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 specific comm
kou commented on code in PR #45424:
URL: https://github.com/apache/arrow/pull/45424#discussion_r1942165470
##
cpp/cmake_modules/ThirdpartyToolchain.cmake:
##
@@ -1256,14 +1256,19 @@ endif()
# - Gandiva has a compile-time (header-only) dependency on Boost, not runtime.
# - Test
kou commented on code in PR #45424:
URL: https://github.com/apache/arrow/pull/45424#discussion_r1942163931
##
cpp/cmake_modules/ThirdpartyToolchain.cmake:
##
@@ -1256,14 +1256,19 @@ endif()
# - Gandiva has a compile-time (header-only) dependency on Boost, not runtime.
# - Test
curioustien commented on PR #45351:
URL: https://github.com/apache/arrow/pull/45351#issuecomment-2635560851
Quick update:
Finding a good way to convert these decimal types and getting all the test
passes take longer than I thought. Probably need a few more days
--
This is an automated
wgtmac commented on code in PR #45424:
URL: https://github.com/apache/arrow/pull/45424#discussion_r1942156943
##
cpp/cmake_modules/ThirdpartyToolchain.cmake:
##
@@ -1256,14 +1256,19 @@ endif()
# - Gandiva has a compile-time (header-only) dependency on Boost, not runtime.
# - T
bionicles commented on issue #6522:
URL: https://github.com/apache/arrow-rs/issues/6522#issuecomment-2635541751
well, you know how it is, a day or two of struggle writing new code saves
15-30 minutes reading the instructions
i'm a major noob with arrow internals, and didn't know all t
phillipleblanc opened a new pull request, #7076:
URL: https://github.com/apache/arrow-rs/pull/7076
# Which issue does this PR close?
Closes #7073
# Rationale for this change
Support for casting large dates from string to Date32.
# What changes are included in this
colin-r-schultz commented on code in PR #45372:
URL: https://github.com/apache/arrow/pull/45372#discussion_r1942123751
##
cpp/src/arrow/record_batch_test.cc:
##
@@ -393,6 +399,27 @@ TEST_F(TestRecordBatch, RemoveColumnEmpty) {
AssertBatchesEqual(*added, *batch1);
}
+TEST_F
kou commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635486651
It seems that
https://cygwin.osuosl.org/noarch/release/tzdata/tzdata-2024a-1.tar.xz removed.
https://github.com/apache/arrow/blob/1567be0d74c5ee90c3d989f6fcefb32c402a736c/ci/scripts/py
kou opened a new pull request, #45424:
URL: https://github.com/apache/arrow/pull/45424
### Rationale for this change
We can use `libarrow_testing.a` without `boost::filesystem` when
`arrow::util::Process` isn't used.
### What changes are included in this PR?
`ARROW_TESTI
github-actions[bot] commented on PR #45424:
URL: https://github.com/apache/arrow/pull/45424#issuecomment-2635479694
Revision: 06a70af5b883ab11fd95a14f50ff0ad11351fd91
Submitted crossbow builds: [ursacomputing/crossbow @
actions-a3bc80b5c0](https://github.com/ursacomputing/crossbow/bra
github-actions[bot] commented on PR #45424:
URL: https://github.com/apache/arrow/pull/45424#issuecomment-2635477709
:warning: GitHub issue #45423 **has been automatically assigned in GitHub**
to PR creator.
--
This is an automated message from the Apache Git Service.
To respond to the mes
kou commented on PR #45424:
URL: https://github.com/apache/arrow/pull/45424#issuecomment-2635477533
@github-actions crossbow submit test-build-cpp-fuzz
--
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 g
phillipleblanc closed pull request #7076: Support converting large dates (i.e.
+10999-12-31) from string to Date32
URL: https://github.com/apache/arrow-rs/pull/7076
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
kou commented on PR #45397:
URL: https://github.com/apache/arrow/pull/45397#issuecomment-2635466674
Oh, sorry. I'll fix it.
#45423
--
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 speci
conbench-apache-arrow[bot] commented on PR #45384:
URL: https://github.com/apache/arrow/pull/45384#issuecomment-2635464268
After merging your PR, Conbench analyzed the 4 benchmarking runs that have
been run so far on merge-commit 1567be0d74c5ee90c3d989f6fcefb32c402a736c.
There were 8
amoeba commented on issue #45156:
URL: https://github.com/apache/arrow/issues/45156#issuecomment-2635424955
Installing a Win10/11 SDK fixes the warning at startup, `--add
Microsoft.VisualStudio.Component.Windows11SDK`.
--
This is an automated message from the Apache Git Service.
To respon
amoeba commented on issue #45156:
URL: https://github.com/apache/arrow/issues/45156#issuecomment-2635366819
I'm looking at this today and have made some progress. My image so far is
only 7.57GB whereas the existing wheel image is 26.8GB which is a nice savings.
So far I have,
```dock
github-actions[bot] commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635356097
Revision: 36e0d50f4c7b51eb4e4147facb6f99da7049609d
Submitted crossbow builds: [ursacomputing/crossbow @
actions-42cab651f2](https://github.com/ursacomputing/crossbow/bra
amoeba commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635349801
@github-actions crossbow submit wheel-windows-cp39-amd64
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
amoeba commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2635349416
I had hoped to reproduce the issue but wasn't able to. I built the image
locally,
```sh
$ $env:DOCKER_BUILDKIT = 0
archery docker build --no-pull python-wheel-windows-vs2019
etseidl commented on issue #7040:
URL: https://github.com/apache/arrow-rs/issues/7040#issuecomment-2635178656
Now I'm conflicted. On the one hand, sentiment in the Parquet community
seems to be heading towards returning an error for data that is malformed as in
this issue. On the other, I'v
WillAyd commented on issue #40220:
URL: https://github.com/apache/arrow/issues/40220#issuecomment-2635181965
FWIW there is a large discussion of this upstream in re2. It doesn't appear
likely that this will be implemented:
https://github.com/google/re2/issues/156
--
This is an auto
pitrou commented on issue #26648:
URL: https://github.com/apache/arrow/issues/26648#issuecomment-2635129385
> happy to! just so I am understanding right: per benchmarking, the
throughput is for `ArrayRangeEqualsSparseUnion` comparisons is about 2x more
efficient?
Yes, roughly.
XiangpengHao commented on issue #7058:
URL: https://github.com/apache/arrow-rs/issues/7058#issuecomment-2635110750
For anyone following along, I think the bug is from flatbuffer, and the
workaround is to turnoff the verifier on receiving the data. The data seems to
be correct, just the veri
lisasgoh commented on issue #7075:
URL: https://github.com/apache/arrow-rs/issues/7075#issuecomment-2635059503
The use case here would be that unlike Datafusion that has a
`register_object_store` method that allows for custom implementations for
object stores, Polars doesn't (https://github
alippai commented on issue #45422:
URL: https://github.com/apache/arrow/issues/45422#issuecomment-2635035677
Is this the equivalent?
```python
BATCH_SIZE = 1024*1024
if len(t.to_batches) > 1:
t = t.combine_chunks()
with pa.OSFile('test3.feather', 'wb') as sink:
wit
WillAyd merged PR #711:
URL: https://github.com/apache/arrow-nanoarrow/pull/711
--
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
kylebarron commented on issue #7075:
URL: https://github.com/apache/arrow-rs/issues/7075#issuecomment-2634984317
It sounds like the OP here is interested in a custom implementation of
`ObjectStore` to serve something other than AWS/GCP/Azure.
That's a bit different than what I've been
tustvold commented on issue #7075:
URL: https://github.com/apache/arrow-rs/issues/7075#issuecomment-2634952585
Tagging @kylebarron.
I think there are a couple of questions here:
* Is the major use-case python interop
* Does a python focused abstraction have more utliity than
tustvold closed issue #5334: Add Attributes API Exposing Broader Set of Object
Metadata
URL: https://github.com/apache/arrow-rs/issues/5334
--
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
tustvold commented on issue #5334:
URL: https://github.com/apache/arrow-rs/issues/5334#issuecomment-2634934529
Closed by https://github.com/apache/arrow-rs/pull/5650
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
scovich commented on issue #6522:
URL: https://github.com/apache/arrow-rs/issues/6522#issuecomment-2634918695
> I prototyped this last month for polars, could share, it's a lot
This seems a bit surprising, given that the feature request is to define new
`num_buffered_rows` and `has_pa
scovich commented on issue #6522:
URL: https://github.com/apache/arrow-rs/issues/6522#issuecomment-2634910042
Perhaps the real concern is about the "bonus" request?
> It would be even nicer if the `parse_json` method could just become part
of either arrow-json or arrow-compute, if parsing
kylebarron commented on issue #5334:
URL: https://github.com/apache/arrow-rs/issues/5334#issuecomment-2634894456
Am I correct that this issue can be closed now that there's
https://docs.rs/object_store/latest/object_store/struct.Attributes.html?
--
This is an automated message from the Ap
himadripal commented on code in PR #7070:
URL: https://github.com/apache/arrow-rs/pull/7070#discussion_r1941766675
##
arrow-cast/src/cast/decimal.rs:
##
@@ -167,7 +167,7 @@ where
let array: PrimitiveArray =
if input_scale == output_scale && input_precision <= outpu
mcshawn10 commented on issue #26648:
URL: https://github.com/apache/arrow/issues/26648#issuecomment-2634831643
> Thank you for your contribution [@mcshawn10](https://github.com/mcshawn10)
!
happy to! just so I am understanding right: per benchmarking, the throughput
is for `ArrayRang
nbauernfeind commented on issue #585:
URL: https://github.com/apache/arrow-java/issues/585#issuecomment-2634733299
Nevermind. While using the returned value from `startNewValue` is more
appropriate, when you start a new value the end offset is initialized to the
start offset. While a little
viirya commented on code in PR #7070:
URL: https://github.com/apache/arrow-rs/pull/7070#discussion_r1941683724
##
arrow-cast/src/cast/decimal.rs:
##
@@ -167,7 +167,7 @@ where
let array: PrimitiveArray =
if input_scale == output_scale && input_precision <= output_pr
WillAyd opened a new pull request, #711:
URL: https://github.com/apache/arrow-nanoarrow/pull/711
(no comment)
--
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
nbauernfeind commented on issue #585:
URL: https://github.com/apache/arrow-java/issues/585#issuecomment-2634698746
Possibly this is the cause of #296.
--
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
lisasgoh opened a new issue, #7075:
URL: https://github.com/apache/arrow-rs/issues/7075
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
The use case here would be to be able to dynamically load libraries with
custom object sto
pitrou commented on issue #26648:
URL: https://github.com/apache/arrow/issues/26648#issuecomment-2634657867
Thank you for your contribution @mcshawn10 !
--
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
pitrou commented on issue #26648:
URL: https://github.com/apache/arrow/issues/26648#issuecomment-2634656884
Issue resolved by pull request 45384
https://github.com/apache/arrow/pull/45384
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
pitrou merged PR #45384:
URL: https://github.com/apache/arrow/pull/45384
--
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
phillipleblanc commented on issue #7073:
URL: https://github.com/apache/arrow-rs/issues/7073#issuecomment-2634612370
take
--
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
phillipleblanc opened a new pull request, #7074:
URL: https://github.com/apache/arrow-rs/pull/7074
# Which issue does this PR close?
Closes #7073
# Rationale for this change
Support for casting large dates from string to Date32.
# What changes are included in this
phillipleblanc opened a new issue, #7073:
URL: https://github.com/apache/arrow-rs/issues/7073
**Describe the bug**
Attempting to convert a string which contains a valid large date in the ISO
format (i.e. `+10999-12-31`) will result in the following error:
`CastError("Cannot cast st
pitrou commented on PR #45384:
URL: https://github.com/apache/arrow/pull/45384#issuecomment-2634533257
It turns out that we didn't have any proper tests for sparse union
comparison, so I added one.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
mcshawn10 commented on code in PR #45384:
URL: https://github.com/apache/arrow/pull/45384#discussion_r1941491449
##
cpp/src/arrow/compare.cc:
##
@@ -381,23 +381,49 @@ class RangeDataEqualsImpl {
const int8_t* right_codes = right_.GetValues(1);
// Unions don't have a
mapsensedev commented on issue #24833:
URL: https://github.com/apache/arrow/issues/24833#issuecomment-2634406031
Any update on the timelines for JS implementation to support buffer
compression
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
pitrou commented on code in PR #45372:
URL: https://github.com/apache/arrow/pull/45372#discussion_r1941468163
##
cpp/src/arrow/record_batch.cc:
##
@@ -102,8 +103,10 @@ class SimpleRecordBatch : public RecordBatch {
std::shared_ptr column(int i) const override {
std::shar
amoeba commented on PR #45420:
URL: https://github.com/apache/arrow/pull/45420#issuecomment-2634390020
My first thought after some searching around was that IANA may have changed
how they issues their TLS cert (which is still possible) and that we just need
a newer version of the certifi pa
1 - 100 of 128 matches
Mail list logo
