github-actions[bot] commented on pull request #7252:
URL: https://github.com/apache/arrow/pull/7252#issuecomment-632986177
https://issues.apache.org/jira/browse/ARROW-8906
This is an automated message from the Apache Git
houqp opened a new pull request #7252:
URL: https://github.com/apache/arrow/pull/7252
This change adds `infer_schema_from_files` function to arrow csv reader
module. Datafusion's `CsvExec` struct is now using this function to do schema
inference from multiple CSV files if needed.
liyafan82 commented on pull request #6425:
URL: https://github.com/apache/arrow/pull/6425#issuecomment-632982040
> Thanks @liyafan82 , looks like they didn't pass on this first try. Any
idea what was causing the error?
>
> ```
> Error accessing files
> Current token
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632975883
I see the problem, I am fixing
This is an automated message from the Apache Git Service.
To respond to the message,
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632975274
Ok I will take a look
This is an automated message from the Apache Git Service.
To respond to the message, please
kou closed pull request #7192:
URL: https://github.com/apache/arrow/pull/7192
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
kou commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632965028
All build errors are fixed.
There are some test failures with MSVC:
https://github.com/apache/arrow/pull/7240/checks?check_run_id=701142871
```text
kou commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632947510
Done.
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
sunchao commented on a change in pull request #7226:
URL: https://github.com/apache/arrow/pull/7226#discussion_r429492355
##
File path: rust/arrow/src/array/builder.rs
##
@@ -1334,6 +1334,57 @@ where
map: HashMap::new(),
}
}
+
+/// Creates a new
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632947370
Note that `kHashSeed` is declared as `uint64_t` so that must be changed also
```
static constexpr uint64_t kHashSeed = 0;
```
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632947188
@kou that's fine, please go ahead and apply it
This is an automated message from the Apache Git Service.
To respond
kou commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632946831
We need the following change for macOS. What do you thin about this change?
```diff
diff --git a/cpp/src/arrow/compute/kernel.cc
b/cpp/src/arrow/compute/kernel.cc
index
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632945792
Perfect thank you :pray:. I'm done hacking on this for now, I'll keep
investigating test failures
This is an
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632945052
I'm just fixing a failing unit test now.
This is an automated message from the Apache Git Service.
To respond to
kou commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632945098
I've fixed a build error with `-DCMAKE_UNITY_BUILD=ON`.
This is an automated message from the Apache Git Service.
To
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632944653
@xhochy @pitrou @kszucs @jorisvandenbossche I just added Python bindings
for this new functionality including generic argument packing and function
dispatching. The new
sunchao edited a comment on pull request #7176:
URL: https://github.com/apache/arrow/pull/7176#issuecomment-632943566
add @sadikovi also who authored this part of code.
This is an automated message from the Apache Git
sunchao commented on pull request #7176:
URL: https://github.com/apache/arrow/pull/7176#issuecomment-632943566
+ @sadikovi also who authored this part of code.
This is an automated message from the Apache Git Service.
To
sunchao edited a comment on pull request #7176:
URL: https://github.com/apache/arrow/pull/7176#issuecomment-632941938
Seems this will be a breaking change as it requires writers to be used in a
different way? and I think the new way is not as intuitive as the old one.
sunchao commented on pull request #7176:
URL: https://github.com/apache/arrow/pull/7176#issuecomment-632941938
Seems this will be a breaking change as it requires writers to be used in a
different way?
This is an automated
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632937311
Done. It looks like GitHub is having some problems today but will keep an
eye on the builds
This is an automated
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632936766
OK, I have it fixed locally. I'm finishing some Python stuff and then I'll
push
This is an automated message from
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632936071
Thank you, I will fix.
This is an automated message from the Apache Git Service.
To respond to the message, please
kou commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632931580
It seems that the added `TestIsInKernel.IsInChunkedArrayInvoke` test is
failed:
```text
[ RUN ] TestIsInKernel.IsInChunkedArrayInvoke
kou commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632930767
Thanks!
I've also done the GLib part.
This is an automated message from the Apache Git Service.
To respond to the
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632929465
@kou done, thank you for catching this, I think it's better to retain this
functionality.
This is an automated
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632927540
The IsIn/Match changes don't look too hard, should be able to push them in
the next half hour or less
This is an
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632925674
I'm taking a look now, I'll report back in a half hour or so
This is an automated message from the Apache Git
wesm edited a comment on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632925336
@kou yes, that was an API change. I can try to restore this functionality
though? It shouldn't be too difficult.
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632925336
@kou yes, that was an API change. I can try to restore this functionality
though?
This is an automated message from
wesm commented on a change in pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#discussion_r429464755
##
File path: cpp/src/arrow/compute/exec.h
##
@@ -0,0 +1,160 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license
wesm commented on a change in pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#discussion_r429464227
##
File path: cpp/src/arrow/compute/kernels/scalar_cast_internal.h
##
@@ -0,0 +1,283 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632919447
Awesome, thanks @kou!
This is an automated message from the Apache Git Service.
To respond to the message, please
wesm commented on a change in pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#discussion_r429463237
##
File path: cpp/src/arrow/compute/kernels/scalar_cast_internal.h
##
@@ -0,0 +1,283 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or
kou commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632919131
I'm working on this now.
It'll be finished in a few hours.
This is an automated message from the Apache Git
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632918704
The R test suite is fixed -- this also turned up some missing test coverage
for things that R was depending on
https://issues.apache.org/jira/browse/ARROW-8895.
I'm going to
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632910238
This problem of kernel dispatch with timestamps that may or may not have
time zones actually brought out a limitation with input type checking. I'm
introducing a simple interface
eerhardt commented on pull request #7246:
URL: https://github.com/apache/arrow/pull/7246#issuecomment-632909076
Also tagging @mr-smidge (sorry I forgot initially)
This is an automated message from the Apache Git Service.
To
nealrichardson commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429433084
##
File path: r/src/arrow_exports.h
##
@@ -0,0 +1,56 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor
nealrichardson commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429431625
##
File path: r/src/arrow_types.h
##
@@ -176,64 +37,21 @@ inline constexpr Rbyte default_value() {
} // namespace Rcpp
-namespace arrow {
BryanCutler edited a comment on pull request #6425:
URL: https://github.com/apache/arrow/pull/6425#issuecomment-632887913
Thanks @liyafan82 , looks like they didn't pass on this first try. Any idea
what was causing the error?
```
Error accessing files
Current token (VALUE_STRING)
BryanCutler commented on pull request #6425:
URL: https://github.com/apache/arrow/pull/6425#issuecomment-632887913
Thanks @liyafan82 , looks like they didn't pass on this first try. Any idea
what was causing the error:
```
Error accessing files
Current token (VALUE_STRING) not
fsaintjacques commented on pull request #7244:
URL: https://github.com/apache/arrow/pull/7244#issuecomment-632842908
@wesm ready to merge.
This is an automated message from the Apache Git Service.
To respond to the message,
fsaintjacques closed pull request #7251:
URL: https://github.com/apache/arrow/pull/7251
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
fsaintjacques commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429395569
##
File path: r/src/arrow_types.h
##
@@ -176,64 +37,21 @@ inline constexpr Rbyte default_value() {
} // namespace Rcpp
-namespace arrow {
sunchao commented on a change in pull request #6770:
URL: https://github.com/apache/arrow/pull/6770#discussion_r429388420
##
File path: rust/datafusion/src/logicalplan.rs
##
@@ -1004,8 +1004,8 @@ mod tests {
.build()?;
let expected = "Projection: #state,
nealrichardson commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429388011
##
File path: r/src/arrow_types.h
##
@@ -176,64 +37,21 @@ inline constexpr Rbyte default_value() {
} // namespace Rcpp
-namespace arrow {
wesm merged pull request #7250:
URL: https://github.com/apache/arrow/pull/7250
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
wesm commented on pull request #7250:
URL: https://github.com/apache/arrow/pull/7250#issuecomment-632826289
+1
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
fsaintjacques edited a comment on pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#issuecomment-632825223
@nealrichardson the next step is probably to break `arrowExport.{h,cpp}`
into one per file cpp file, e.g.
```
src/buffer.cpp
src/buffer.h
fsaintjacques commented on pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#issuecomment-632825223
@nealrichardson the next step is probably to break `arrowExport.{h,cpp}`
into one per file cpp file, e.g.
```
src/buffer.cpp
src/buffer.h
wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-632824231
I just rebased and pushed MSVC build fixes (that works for me at least
locally on VS 2017). I'm going to fix the R failure next and then address the
other accumulated comments
sunchao commented on pull request #6935:
URL: https://github.com/apache/arrow/pull/6935#issuecomment-632823350
Merged. Thanks @rdettai !
This is an automated message from the Apache Git Service.
To respond to the message,
sunchao closed pull request #6935:
URL: https://github.com/apache/arrow/pull/6935
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
fsaintjacques commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429372721
##
File path: r/src/arrow_types.h
##
@@ -176,64 +37,21 @@ inline constexpr Rbyte default_value() {
} // namespace Rcpp
-namespace arrow {
fsaintjacques commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429372456
##
File path: r/src/arrow_types.h
##
@@ -176,64 +37,21 @@ inline constexpr Rbyte default_value() {
} // namespace Rcpp
-namespace arrow {
fsaintjacques commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429372456
##
File path: r/src/arrow_types.h
##
@@ -176,64 +37,21 @@ inline constexpr Rbyte default_value() {
} // namespace Rcpp
-namespace arrow {
fsaintjacques commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429371319
##
File path: r/src/arrow_exports.h
##
@@ -0,0 +1,56 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor
wesm commented on pull request #7213:
URL: https://github.com/apache/arrow/pull/7213#issuecomment-632819189
I just stumbled on failures on MSVC in these benchmarks (see
https://issues.apache.org/jira/browse/ARROW-8892). I'm fixing the immediate
issues that I found but the problem of
fsaintjacques commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429369559
##
File path: r/src/array_from_vector.cpp
##
@@ -20,6 +20,11 @@
#include "./arrow_types.h"
#if defined(ARROW_R_WITH_ARROW)
+#include
+#include
fsaintjacques commented on pull request #7251:
URL: https://github.com/apache/arrow/pull/7251#issuecomment-632774031
It was [failing](https://github.com/apache/arrow/runs/699629364) on master,
weirdly, the autotune commit didn't lunch any build.
zgramana commented on pull request #7246:
URL: https://github.com/apache/arrow/pull/7246#issuecomment-632754359
This is helpful, thanks @eerhardt! Editorconfig is preferable for those of
us using VS Code on *nix.
This is an
nealrichardson commented on a change in pull request #7245:
URL: https://github.com/apache/arrow/pull/7245#discussion_r429303750
##
File path: r/src/arrow_types.h
##
@@ -176,64 +37,21 @@ inline constexpr Rbyte default_value() {
} // namespace Rcpp
-namespace arrow {
github-actions[bot] commented on pull request #7251:
URL: https://github.com/apache/arrow/pull/7251#issuecomment-632716197
https://issues.apache.org/jira/browse/ARROW-8890
This is an automated message from the Apache Git
paddyhoran commented on pull request #6770:
URL: https://github.com/apache/arrow/pull/6770#issuecomment-632715411
Sorry @maxburke, I'm snowed under at work at the moment. Once @sunchao or
@sadikovi approves we are good, they are the Parquet experts.
fsaintjacques opened a new pull request #7251:
URL: https://github.com/apache/arrow/pull/7251
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
fsaintjacques commented on a change in pull request #7030:
URL: https://github.com/apache/arrow/pull/7030#discussion_r426869642
##
File path:
java/dataset/src/main/java/org/apache/arrow/dataset/jni/JniLoader.java
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software
github-actions[bot] commented on pull request #7250:
URL: https://github.com/apache/arrow/pull/7250#issuecomment-632705636
Thanks for opening a pull request!
Could you open an issue for this pull request on JIRA?
https://issues.apache.org/jira/browse/ARROW
Then
wesm opened a new pull request #7250:
URL: https://github.com/apache/arrow/pull/7250
The RC verification script fails without this.
This is an automated message from the Apache Git Service.
To respond to the message, please
fsaintjacques closed pull request #7249:
URL: https://github.com/apache/arrow/pull/7249
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
fsaintjacques commented on pull request #7249:
URL: https://github.com/apache/arrow/pull/7249#issuecomment-632696516
The lint failure was introduced by me, ignore.
This is an automated message from the Apache Git Service.
To
github-actions[bot] commented on pull request #7249:
URL: https://github.com/apache/arrow/pull/7249#issuecomment-632673037
https://issues.apache.org/jira/browse/ARROW-8889
This is an automated message from the Apache Git
liyafan82 commented on pull request #7149:
URL: https://github.com/apache/arrow/pull/7149#issuecomment-632669016
@wesm Thanks a lot for your effort, and sorry for my late response.
Thank all reviewers for the good comments.
lidavidm opened a new pull request #7249:
URL: https://github.com/apache/arrow/pull/7249
This avoids passing an invalid (null) reference to C++.
This is an automated message from the Apache Git Service.
To respond to the
andygrove commented on pull request #4140:
URL: https://github.com/apache/arrow/pull/4140#issuecomment-632665136
Sorry, I didn't get to this yet. I have learned how to wrangle these release
scripts now so will address this today or over the weekend.
fsaintjacques closed pull request #7245:
URL: https://github.com/apache/arrow/pull/7245
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
fsaintjacques commented on pull request #7244:
URL: https://github.com/apache/arrow/pull/7244#issuecomment-632657355
Spurious s3 failure.
This is an automated message from the Apache Git Service.
To respond to the message,
liyafan82 commented on pull request #6425:
URL: https://github.com/apache/arrow/pull/6425#issuecomment-632603791
@BryanCutler @emkornfield Sorry for my late response.
I have removed the skip. Let's see if the integration tests can pass this
time.
github-actions[bot] commented on pull request #7248:
URL: https://github.com/apache/arrow/pull/7248#issuecomment-632542028
https://issues.apache.org/jira/browse/ARROW-8402
This is an automated message from the Apache Git
liyafan82 opened a new pull request #7248:
URL: https://github.com/apache/arrow/pull/7248
In this PR, we provide the validateFull functionalities, and refactor the
validate logic. We separate the validation logic into 3 visitors:
1. Type validation: validate vector type information
pravindra closed pull request #7247:
URL: https://github.com/apache/arrow/pull/7247
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
81 matches
Mail list logo