Jimexist opened a new issue #1232:
URL: https://github.com/apache/arrow-datafusion/issues/1232
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
add feature to allow datafusion cli to list columns from a table
**Describe
Jimexist opened a new pull request #1231:
URL: https://github.com/apache/arrow-datafusion/pull/1231
# Which issue does this PR close?
Closes #.
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing changes
Jimexist merged pull request #1229:
URL: https://github.com/apache/arrow-datafusion/pull/1229
--
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: gith
Jimexist closed issue #1230:
URL: https://github.com/apache/arrow-datafusion/issues/1230
--
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-un
Dandandan commented on issue #700:
URL:
https://github.com/apache/arrow-datafusion/issues/700#issuecomment-958685830
AFAIK No @mingmwang . Feel free to work on this - that would be great.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
vvellanki commented on a change in pull request #11180:
URL: https://github.com/apache/arrow/pull/11180#discussion_r741641131
##
File path: cpp/src/gandiva/gdv_function_stubs.cc
##
@@ -794,6 +794,56 @@ const char* gdv_fn_initcap_utf8(int64_t context, const
char* data, int32_t
vvellanki commented on a change in pull request #11287:
URL: https://github.com/apache/arrow/pull/11287#discussion_r741638784
##
File path: cpp/src/gandiva/function_registry_string.cc
##
@@ -406,6 +406,10 @@ std::vector GetStringFunctionRegistry() {
NativeFunction("spl
xudong963 edited a comment on pull request #1202:
URL: https://github.com/apache/arrow-datafusion/pull/1202#issuecomment-958674138
The remaining LGTM, wait for @alamb to review. Nice work @capkurmagati
--
This is an automated message from the Apache Git Service.
To respond to the message
xudong963 commented on pull request #1202:
URL: https://github.com/apache/arrow-datafusion/pull/1202#issuecomment-958674138
The remaining LGTM, wait for @alamb to review.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
vvellanki commented on a change in pull request #11287:
URL: https://github.com/apache/arrow/pull/11287#discussion_r741632047
##
File path: cpp/src/gandiva/gdv_function_stubs.cc
##
@@ -794,6 +794,27 @@ const char* gdv_fn_initcap_utf8(int64_t context, const
char* data, int32_t
cyb70289 commented on pull request #11588:
URL: https://github.com/apache/arrow/pull/11588#issuecomment-958669125
Thanks @niyue !
I did a similar test to read randomly at most 1/4 pages of a memory mapped
file with 1G size. Without `madvise`, almost all 1G data is in the page cache
afte
xudong963 commented on a change in pull request #1202:
URL: https://github.com/apache/arrow-datafusion/pull/1202#discussion_r741628302
##
File path: datafusion/src/logical_plan/expr.rs
##
@@ -1935,10 +1935,32 @@ fn create_name(e: &Expr, input_schema: &DFSchema) ->
Result {
bkmgit commented on a change in pull request #11205:
URL: https://github.com/apache/arrow/pull/11205#discussion_r741627611
##
File path: docs/source/developers/cpp/building.rst
##
@@ -41,6 +41,9 @@ Building requires:
sufficient. For Windows, at least Visual Studio 2017 is re
xudong963 commented on a change in pull request #1202:
URL: https://github.com/apache/arrow-datafusion/pull/1202#discussion_r741626999
##
File path: datafusion/src/physical_plan/planner.rs
##
@@ -176,6 +176,21 @@ fn create_physical_name(e: &Expr, is_first_expr: bool) ->
Result
xudong963 commented on a change in pull request #1202:
URL: https://github.com/apache/arrow-datafusion/pull/1202#discussion_r741626089
##
File path: datafusion/src/logical_plan/expr.rs
##
@@ -1935,10 +1935,32 @@ fn create_name(e: &Expr, input_schema: &DFSchema) ->
Result {
rkavanap commented on a change in pull request #11051:
URL: https://github.com/apache/arrow/pull/11051#discussion_r741622617
##
File path: cpp/src/gandiva/precompiled/string_ops.cc
##
@@ -1642,6 +1642,55 @@ const char* convert_toUTF8(int64_t context, const char*
value, int32_t
xudong963 commented on pull request #1135:
URL: https://github.com/apache/arrow-datafusion/pull/1135#issuecomment-958658978
> Am I correct that in the current design, the `null_equal_safe` argument
will apply to all join key pairs? i.e. will we be able to have `t1 JOIN t2 ON
t1.col1 = t2.c
rkavanap commented on a change in pull request #11051:
URL: https://github.com/apache/arrow/pull/11051#discussion_r741622617
##
File path: cpp/src/gandiva/precompiled/string_ops.cc
##
@@ -1642,6 +1642,55 @@ const char* convert_toUTF8(int64_t context, const char*
value, int32_t
Jimexist opened a new issue #1230:
URL: https://github.com/apache/arrow-datafusion/issues/1230
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
datafusion cli to add list tables command
**Describe the solution you'd like
Jimexist opened a new pull request #1229:
URL: https://github.com/apache/arrow-datafusion/pull/1229
# Which issue does this PR close?
Closes #.
# Rationale for this change
add list table command
# What changes are included in this PR?
# Are there any u
Jimexist merged pull request #1224:
URL: https://github.com/apache/arrow-datafusion/pull/1224
--
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: gith
Jimexist closed issue #1216:
URL: https://github.com/apache/arrow-datafusion/issues/1216
--
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-un
capkurmagati commented on a change in pull request #1202:
URL: https://github.com/apache/arrow-datafusion/pull/1202#discussion_r741610021
##
File path: datafusion/tests/sql.rs
##
@@ -5305,3 +5305,57 @@ async fn case_with_bool_type_result() -> Result<()> {
assert_eq!(expect
capkurmagati commented on pull request #1202:
URL: https://github.com/apache/arrow-datafusion/pull/1202#issuecomment-958643103
@xudong963 @alamb Thanks for the advice. I added some tests for the expr.
PTAL.
I will rebase the branch after reflecting your reviews.
--
This is an automate
houqp commented on issue #1228:
URL:
https://github.com/apache/arrow-datafusion/issues/1228#issuecomment-958640365
@alamb if I recall correctly, the partialOrd for Expr as added to help with
unit testing right?
--
This is an automated message from the Apache Git Service.
To respond to t
xudong963 commented on issue #1228:
URL:
https://github.com/apache/arrow-datafusion/issues/1228#issuecomment-958638881
Let me know your thoughts ❤️ @houqp @alamb @Jimexist @Dandandan
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
xudong963 opened a new issue #1228:
URL: https://github.com/apache/arrow-datafusion/issues/1228
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
During I try to implement `Select (subquery)` , I add `
Select(Box)` in `Expr`. Becau
houqp edited a comment on pull request #1135:
URL: https://github.com/apache/arrow-datafusion/pull/1135#issuecomment-958638468
Am I correct that in the current design, the `null_equal_safe` argument will
apply to all join key pairs? i.e. will we be able to have `t1 JOIN t2 ON
t1.col1 = t2.
houqp commented on pull request #1135:
URL: https://github.com/apache/arrow-datafusion/pull/1135#issuecomment-958638468
Am I correct that in the current design, the `null_equal_safe` argument will
apply to all join key pairs? i.e. will we be able to have `JOIN ON t1.col1 =
t2.col1, t1.col2
Jimexist opened a new issue #1227:
URL: https://github.com/apache/arrow-datafusion/issues/1227
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
A challenge to add function searching and listing in datafusion cli
**Descri
Jimexist opened a new issue #1226:
URL: https://github.com/apache/arrow-datafusion/issues/1226
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
a challenge to add support for interrupt signal, to clear current line
instead of
Jimexist opened a new pull request #1225:
URL: https://github.com/apache/arrow-datafusion/pull/1225
# Which issue does this PR close?
Closes #.
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing changes
westonpace commented on pull request #11486:
URL: https://github.com/apache/arrow/pull/11486#issuecomment-958617799
Hmm, I'll have to take a look and see what's up there. Some other ways to
run lint are `ninja lint` and `archery lint --cpplint`. The former requires
you to use the ninja g
niyue commented on pull request #11486:
URL: https://github.com/apache/arrow/pull/11486#issuecomment-958615422
> Looks like one last CI formatting thing:
>
> ```
> /arrow/cpp/src/arrow/ipc/reader_internal.h:84: Could not find a newline
character at the end of the file. [whitespa
niyue edited a comment on pull request #11588:
URL: https://github.com/apache/arrow/pull/11588#issuecomment-958612133
> Thanks for the PR.
> I'm not sure if exposing madvise options is beneficial. If we expose
`RANDOM`, then why not `SEQUENTIAL`?
> I prefer don't expose these access p
niyue commented on pull request #11588:
URL: https://github.com/apache/arrow/pull/11588#issuecomment-958612133
> Thanks for the PR.
> I'm not sure if exposing madvise options is beneficial. If we expose
`RANDOM`, then why not `SEQUENTIAL`?
> I prefer don't expose these access pattern
ursabot edited a comment on pull request #11585:
URL: https://github.com/apache/arrow/pull/11585#issuecomment-958210366
Benchmark runs are scheduled for baseline =
92e3da573738d21ef7e74775f67ede499d59ebd7 and contender =
bf67ec74635db2183619601f025e4724bd5a6b75.
bf67ec74635db2183619601f02
westonpace commented on pull request #11486:
URL: https://github.com/apache/arrow/pull/11486#issuecomment-958609714
Looks like one last CI formatting thing:
```
/arrow/cpp/src/arrow/ipc/reader_internal.h:84: Could not find a newline
character at the end of the file. [whitespace/
cyb70289 edited a comment on pull request #11588:
URL: https://github.com/apache/arrow/pull/11588#issuecomment-958605062
Thanks for the PR.
I'm not sure if exposing madvise options is beneficial. If we expose
`RANDOM`, then why not `SEQUENTIAL`?
I prefer don't expose these access patt
Jimexist opened a new pull request #1224:
URL: https://github.com/apache/arrow-datafusion/pull/1224
# Which issue does this PR close?
Closes #1216
# Rationale for this change
add \q as quit command and add \? for help
# What changes are included in this PR?
cyb70289 commented on pull request #11588:
URL: https://github.com/apache/arrow/pull/11588#issuecomment-958605062
Thanks for the PR.
I'm not sure if exposing madvise options is beneficial. If we expose
`RANDOM`, then why not `SEQUENTIAL`?
I prefer don't expose these access pattern rel
westonpace commented on a change in pull request #11542:
URL: https://github.com/apache/arrow/pull/11542#discussion_r74154
##
File path: cpp/src/arrow/compute/kernels/vector_buffer_test.cc
##
@@ -0,0 +1,382 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+
westonpace commented on a change in pull request #11542:
URL: https://github.com/apache/arrow/pull/11542#discussion_r741577703
##
File path: cpp/src/arrow/compute/kernels/vector_buffer.cc
##
@@ -0,0 +1,345 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or
ianmcook commented on a change in pull request #11023:
URL: https://github.com/apache/arrow/pull/11023#discussion_r741576370
##
File path: r/tests/testthat/test-dplyr-funcs-string.R
##
@@ -467,6 +467,18 @@ test_that("strsplit and str_split", {
)
})
+test_that("strrep", {
yahoNanJing commented on a change in pull request #1062:
URL: https://github.com/apache/arrow-datafusion/pull/1062#discussion_r741575976
##
File path: datafusion/src/datasource/object_store/hdfs/os_parquet.rs
##
@@ -0,0 +1,102 @@
+// Licensed to the Apache Software Foundation (
yahoNanJing commented on a change in pull request #1062:
URL: https://github.com/apache/arrow-datafusion/pull/1062#discussion_r741575479
##
File path: datafusion/src/datasource/object_store/hdfs/os_parquet.rs
##
@@ -0,0 +1,102 @@
+// Licensed to the Apache Software Foundation (
ianmcook commented on a change in pull request #11023:
URL: https://github.com/apache/arrow/pull/11023#discussion_r741575436
##
File path: r/R/expression.R
##
@@ -100,7 +100,8 @@
# use `%/%` above.
"%%" = "divide_checked",
"^" = "power_checked",
- "%in%" = "is_in_meta
yahoNanJing opened a new pull request #1223:
URL: https://github.com/apache/arrow-datafusion/pull/1223
# Which issue does this PR close?
Closes #1060. It's a refactor version of PR #1062.
# Rationale for this change
Currently, we can only read parquet files from
westonpace commented on a change in pull request #11542:
URL: https://github.com/apache/arrow/pull/11542#discussion_r741573603
##
File path: cpp/src/arrow/compute/kernels/vector_buffer.cc
##
@@ -0,0 +1,345 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or
ursabot edited a comment on pull request #11585:
URL: https://github.com/apache/arrow/pull/11585#issuecomment-958210366
Benchmark runs are scheduled for baseline =
92e3da573738d21ef7e74775f67ede499d59ebd7 and contender =
bf67ec74635db2183619601f025e4724bd5a6b75.
bf67ec74635db2183619601f02
kingeasternsun opened a new pull request #909:
URL: https://github.com/apache/arrow-rs/pull/909
# Which issue does this PR close?
Closes #908
# Rationale for this change
# What changes are included in this PR?
fix the document li
kingeasternsun opened a new issue #908:
URL: https://github.com/apache/arrow-rs/issues/908
**Describe the bug**
A clear and concise description of what the bug is.
the SerializedFileReader has moved from reader.rs to serialized_reader.rs ,
but document of mod.rs not updated
yahoNanJing commented on issue #1210:
URL:
https://github.com/apache/arrow-datafusion/issues/1210#issuecomment-958580781
It's fixed from the root cause by #1141.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
yahoNanJing closed issue #1210:
URL: https://github.com/apache/arrow-datafusion/issues/1210
--
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
yahoNanJing closed pull request #1211:
URL: https://github.com/apache/arrow-datafusion/pull/1211
--
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: g
yahoNanJing commented on pull request #1211:
URL: https://github.com/apache/arrow-datafusion/pull/1211#issuecomment-958580503
Hi @houqp, thanks to #1141. It fixed the root cause. We can close this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message,
westonpace commented on a change in pull request #11556:
URL: https://github.com/apache/arrow/pull/11556#discussion_r741557476
##
File path: cpp/src/arrow/dataset/dataset_writer.cc
##
@@ -83,128 +84,163 @@ class Throttle {
std::mutex mutex_;
};
+struct DatasetWriterState
westonpace commented on a change in pull request #11556:
URL: https://github.com/apache/arrow/pull/11556#discussion_r741556380
##
File path: cpp/src/arrow/table.h
##
@@ -208,6 +208,15 @@ class ARROW_EXPORT Table {
Result> CombineChunks(
MemoryPool* pool = default_memo
westonpace commented on a change in pull request #11556:
URL: https://github.com/apache/arrow/pull/11556#discussion_r741556138
##
File path: cpp/src/arrow/util/async_util.h
##
@@ -176,6 +176,13 @@ class ARROW_EXPORT SerializedAsyncTaskGroup {
/// The returned future that wil
westonpace commented on a change in pull request #11556:
URL: https://github.com/apache/arrow/pull/11556#discussion_r741556010
##
File path: cpp/src/arrow/util/async_util.h
##
@@ -176,6 +176,13 @@ class ARROW_EXPORT SerializedAsyncTaskGroup {
/// The returned future that wil
ursabot edited a comment on pull request #11587:
URL: https://github.com/apache/arrow/pull/11587#issuecomment-958141990
Benchmark runs are scheduled for baseline =
2917baf4744940ebe809c5b16effc806859f8843 and contender =
92e3da573738d21ef7e74775f67ede499d59ebd7.
92e3da573738d21ef7e74775f6
mingmwang commented on issue #700:
URL:
https://github.com/apache/arrow-datafusion/issues/700#issuecomment-958525993
Is there any PR related to this ? If not I think I can work on this.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
michalursa commented on a change in pull request #11446:
URL: https://github.com/apache/arrow/pull/11446#discussion_r741544566
##
File path: cpp/src/arrow/compute/exec/hash_join_dict.h
##
@@ -0,0 +1,321 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or mo
ursabot edited a comment on pull request #11587:
URL: https://github.com/apache/arrow/pull/11587#issuecomment-958141990
Benchmark runs are scheduled for baseline =
2917baf4744940ebe809c5b16effc806859f8843 and contender =
92e3da573738d21ef7e74775f67ede499d59ebd7.
92e3da573738d21ef7e74775f6
cpcloud commented on pull request #11466:
URL: https://github.com/apache/arrow/pull/11466#issuecomment-958419705
I think making these kernels would probably be a good idea, if that allows
us to move some of the logic into smaller functions.
Unions seem like a great follow up, and wel
cpcloud commented on a change in pull request #11466:
URL: https://github.com/apache/arrow/pull/11466#discussion_r741531580
##
File path: cpp/src/arrow/compute/exec/expression.h
##
@@ -112,7 +113,7 @@ class ARROW_EXPORT Expression {
// post-bind properties
ValueDesc
cpcloud commented on a change in pull request #11466:
URL: https://github.com/apache/arrow/pull/11466#discussion_r741531506
##
File path: cpp/src/arrow/compute/exec/expression.cc
##
@@ -512,7 +511,31 @@ Result ExecuteScalarExpression(const Expression&
expr, const ExecBatch& i
cpcloud commented on a change in pull request #11466:
URL: https://github.com/apache/arrow/pull/11466#discussion_r741531355
##
File path: cpp/src/arrow/array/array_nested.cc
##
@@ -541,56 +541,62 @@ std::shared_ptr StructArray::GetFieldByName(const
std::string& name) cons
R
lidavidm commented on a change in pull request #11452:
URL: https://github.com/apache/arrow/pull/11452#discussion_r741528938
##
File path: cpp/src/arrow/compute/kernels/hash_aggregate.cc
##
@@ -1677,6 +1703,177 @@ struct GroupedMinMaxImpl final : public
GroupedAggregator {
lidavidm commented on a change in pull request #11452:
URL: https://github.com/apache/arrow/pull/11452#discussion_r741528772
##
File path: cpp/src/arrow/compute/kernels/hash_aggregate.cc
##
@@ -1677,6 +1703,177 @@ struct GroupedMinMaxImpl final : public
GroupedAggregator {
edponce commented on a change in pull request #11452:
URL: https://github.com/apache/arrow/pull/11452#discussion_r741473297
##
File path: cpp/src/arrow/compute/kernels/hash_aggregate.cc
##
@@ -603,8 +603,9 @@ struct GroupedValueTraits {
};
template
-void VisitGroupedValues
niyue commented on a change in pull request #11486:
URL: https://github.com/apache/arrow/pull/11486#discussion_r741521123
##
File path: cpp/src/arrow/ipc/message.h
##
@@ -441,6 +441,10 @@ class ARROW_EXPORT MessageReader {
virtual Result> ReadNextMessage() = 0;
};
+// the
niyue commented on a change in pull request #11486:
URL: https://github.com/apache/arrow/pull/11486#discussion_r741520738
##
File path: cpp/src/arrow/ipc/reader_internal.h
##
@@ -0,0 +1,82 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor
ursabot edited a comment on pull request #11586:
URL: https://github.com/apache/arrow/pull/11586#issuecomment-958141322
Benchmark runs are scheduled for baseline =
7667c10c448777b4b11dd88a084fa53e1da3e7c6 and contender =
2917baf4744940ebe809c5b16effc806859f8843.
2917baf4744940ebe809c5b16e
ursabot edited a comment on pull request #11586:
URL: https://github.com/apache/arrow/pull/11586#issuecomment-958141322
Benchmark runs are scheduled for baseline =
7667c10c448777b4b11dd88a084fa53e1da3e7c6 and contender =
2917baf4744940ebe809c5b16effc806859f8843.
2917baf4744940ebe809c5b16e
simpl1g commented on a change in pull request #11584:
URL: https://github.com/apache/arrow/pull/11584#discussion_r741500686
##
File path: ruby/red-arrow/doc/text/examples.md
##
@@ -0,0 +1,106 @@
+
+
+# Examples
+
+## Create table
+### From file
+```ruby
+table = Arrow::Table.lo
ursabot edited a comment on pull request #11585:
URL: https://github.com/apache/arrow/pull/11585#issuecomment-958210366
Benchmark runs are scheduled for baseline =
92e3da573738d21ef7e74775f67ede499d59ebd7 and contender =
bf67ec74635db2183619601f025e4724bd5a6b75.
bf67ec74635db2183619601f02
ursabot commented on pull request #11585:
URL: https://github.com/apache/arrow/pull/11585#issuecomment-958210366
Benchmark runs are scheduled for baseline =
92e3da573738d21ef7e74775f67ede499d59ebd7 and contender =
bf67ec74635db2183619601f025e4724bd5a6b75.
bf67ec74635db2183619601f025e4724b
nealrichardson closed pull request #11585:
URL: https://github.com/apache/arrow/pull/11585
--
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-
coryan commented on pull request #11569:
URL: https://github.com/apache/arrow/pull/11569#issuecomment-958207722
Ping? Is there something else I need to do here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
lidavidm commented on pull request #11507:
URL: https://github.com/apache/arrow/pull/11507#issuecomment-958183255
Alright - I can replicate it now, on Ubuntu 20.04:
This is _without_ any patches - so I think the problem runs even deeper than
we expected. I'll do my best to find a so
emkornfield commented on pull request #11591:
URL: https://github.com/apache/arrow/pull/11591#issuecomment-957998681
One more note. The JIRA linked is about adjusting the compression level
used by compression algorithms (trading off speed for output size). I didnt
notice a change here fo
ianmcook commented on a change in pull request #11592:
URL: https://github.com/apache/arrow/pull/11592#discussion_r741201435
##
File path: r/R/dplyr-functions.R
##
@@ -848,6 +848,21 @@ nse_funcs$wday <- function(x,
Expression$create("day_of_week", x, options = list(count_fro
paleolimbot commented on pull request #11592:
URL: https://github.com/apache/arrow/pull/11592#issuecomment-957997188
Ok! I think this is cleaner...I had incorrectly included TIME32 and TIME64
with TIMESTAMP as well. Does DATE64 map back to POSIXct?
I think it's best to leave POSIXlt
lidavidm commented on a change in pull request #11023:
URL: https://github.com/apache/arrow/pull/11023#discussion_r741054324
##
File path: cpp/src/arrow/compute/kernels/scalar_arithmetic_test.cc
##
@@ -20,6 +20,7 @@
#include
#include
#include
+#include
Review comment:
bkietz commented on a change in pull request #11384:
URL: https://github.com/apache/arrow/pull/11384#discussion_r741160879
##
File path: cpp/cmake_modules/BuildUtils.cmake
##
@@ -723,22 +724,27 @@ function(ADD_TEST_CASE REL_TEST_NAME)
add_dependencies(${TEST_NAME} ${ARG_EX
jonkeane commented on a change in pull request #11592:
URL: https://github.com/apache/arrow/pull/11592#discussion_r741269486
##
File path: r/R/dplyr-functions.R
##
@@ -848,6 +848,21 @@ nse_funcs$wday <- function(x,
Expression$create("day_of_week", x, options = list(count_fro
kou closed pull request #11587:
URL: https://github.com/apache/arrow/pull/11587
--
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...
jonkeane commented on pull request #11592:
URL: https://github.com/apache/arrow/pull/11592#issuecomment-957908069
--
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
alamb commented on pull request #1204:
URL: https://github.com/apache/arrow-datafusion/pull/1204#issuecomment-958128372
--
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 u
alamb commented on pull request #1212:
URL: https://github.com/apache/arrow-datafusion/pull/1212#issuecomment-958114854
I think that during planning we should treat the type of any
`ScalarValue::*(None)` as `DataType::Null` and coerce them appropriately to the
needed types as part of physi
ursabot commented on pull request #11587:
URL: https://github.com/apache/arrow/pull/11587#issuecomment-958141990
Benchmark runs are scheduled for baseline =
2917baf4744940ebe809c5b16effc806859f8843 and contender =
92e3da573738d21ef7e74775f67ede499d59ebd7.
92e3da573738d21ef7e74775f67ede499
alamb closed issue #614:
URL: https://github.com/apache/arrow-datafusion/issues/614
--
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-unsubsc
alamb closed issue #1162:
URL: https://github.com/apache/arrow-datafusion/issues/1162
--
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-unsub
houqp merged pull request #873:
URL: https://github.com/apache/arrow-datafusion/pull/873
--
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-un
alamb commented on a change in pull request #1062:
URL: https://github.com/apache/arrow-datafusion/pull/1062#discussion_r740941986
##
File path: datafusion/src/datasource/object_store/hdfs/os_parquet.rs
##
@@ -0,0 +1,102 @@
+// Licensed to the Apache Software Foundation (ASF) u
codecov-commenter commented on pull request #907:
URL: https://github.com/apache/arrow-rs/pull/907#issuecomment-958134744
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/907?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+A
edponce commented on issue #11559:
URL: https://github.com/apache/arrow/issues/11559#issuecomment-958150821
@teneon If these solutions worked for you, please close this issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
emkornfield commented on a change in pull request #11591:
URL: https://github.com/apache/arrow/pull/11591#discussion_r741330445
##
File path:
java/compression/src/test/java/org/apache/arrow/compression/TestCompressionCodecFile.java
##
@@ -0,0 +1,176 @@
+/*
+ * Licensed to the
alamb commented on pull request #521:
URL: https://github.com/apache/arrow-rs/pull/521#issuecomment-958149214
I am going through old PRs and this one seems stalled. I am wondering what
we would like to do with this one? Is ok to merge? Are we doing an alternate
implementation? Do we have s
1 - 100 of 665 matches
Mail list logo
