voltcode opened a new issue #464:
URL: https://github.com/apache/arrow-datafusion/issues/464
I browsed the readme and slides but failed to grok - can DataFusion handle
larger than RAM datasets? In other words, if I register multiple parquet files,
which size exceeds RAM, will they get all
Jimexist opened a new pull request #463:
URL: https://github.com/apache/arrow-datafusion/pull/463
# Which issue does this PR close?
Closes #.
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing changes?
Dandandan commented on a change in pull request #384:
URL: https://github.com/apache/arrow-rs/pull/384#discussion_r642802751
##
File path: parquet/src/util/memory.rs
##
@@ -292,19 +292,28 @@ impl BufferPtr {
}
/// Returns slice of data in this buffer.
+#[inline]
cyb70289 commented on pull request #10364:
URL: https://github.com/apache/arrow/pull/10364#issuecomment-851806478
> Please extract the decimal upscaling from the addition kernel into an
implicit cast. This will simplify the addition kernel to stateless addition
(IIUC) and give callers cont
nirandaperera commented on a change in pull request #10410:
URL: https://github.com/apache/arrow/pull/10410#discussion_r642728158
##
File path: cpp/src/arrow/compute/kernels/scalar_if_else_test.cc
##
@@ -0,0 +1,474 @@
+// Licensed to the Apache Software Foundation (ASF) under o
nirandaperera commented on pull request #10410:
URL: https://github.com/apache/arrow/pull/10410#issuecomment-851743321
@github-actions autotune
--
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 th
codecov-commenter edited a comment on pull request #353:
URL: https://github.com/apache/arrow-rs/pull/353#issuecomment-848275425
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/353?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_ter
alippai commented on pull request #353:
URL: https://github.com/apache/arrow-rs/pull/353#issuecomment-851701049
@ritchie46 updated the API with correct name + unsafe, also added benchmark.
```
bench_string/bench_string
codecov-commenter edited a comment on pull request #459:
URL: https://github.com/apache/arrow-datafusion/pull/459#issuecomment-851638878
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/459?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+
codecov-commenter edited a comment on pull request #384:
URL: https://github.com/apache/arrow-rs/pull/384#issuecomment-851063613
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/384?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_ter
Dandandan commented on a change in pull request #68:
URL: https://github.com/apache/arrow-datafusion/pull/68#discussion_r642571782
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -339,6 +325,36 @@ pin_project! {
}
}
+fn hash_(group_values: &[ArrayRef]
Dandandan commented on a change in pull request #436:
URL: https://github.com/apache/arrow-datafusion/pull/436#discussion_r642675738
##
File path: datafusion/src/optimizer/remove_duplicate_filters.rs
##
@@ -0,0 +1,611 @@
+// regarding copyright ownership. The ASF licenses this
Dandandan commented on a change in pull request #436:
URL: https://github.com/apache/arrow-datafusion/pull/436#discussion_r642675738
##
File path: datafusion/src/optimizer/remove_duplicate_filters.rs
##
@@ -0,0 +1,611 @@
+// regarding copyright ownership. The ASF licenses this
Dandandan commented on issue #451:
URL:
https://github.com/apache/arrow-datafusion/issues/451#issuecomment-851679821
Yeah I believe joins are reasonably fast currently. I do need to do some
comparisions (e.g. add the join queries to
https://github.com/h2oai/db-benchmark/pull/182)
T
codecov-commenter edited a comment on pull request #441:
URL: https://github.com/apache/arrow-datafusion/pull/441#issuecomment-850900230
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/441?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+
Dandandan opened a new issue #462:
URL: https://github.com/apache/arrow-datafusion/issues/462
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
Recursive CTEs are interesting to support more complex algorithms like graph
processing
Dandandan commented on a change in pull request #436:
URL: https://github.com/apache/arrow-datafusion/pull/436#discussion_r642668210
##
File path: datafusion/src/optimizer/mod.rs
##
@@ -25,4 +25,5 @@ pub mod hash_build_probe_order;
pub mod limit_push_down;
pub mod optimizer;
Dandandan commented on a change in pull request #436:
URL: https://github.com/apache/arrow-datafusion/pull/436#discussion_r642667853
##
File path: datafusion/src/optimizer/remove_duplicate_filters.rs
##
@@ -0,0 +1,611 @@
+// regarding copyright ownership. The ASF licenses this
Dandandan commented on a change in pull request #436:
URL: https://github.com/apache/arrow-datafusion/pull/436#discussion_r642667063
##
File path: datafusion/src/optimizer/remove_duplicate_filters.rs
##
@@ -0,0 +1,611 @@
+// regarding copyright ownership. The ASF licenses this
Dandandan commented on a change in pull request #436:
URL: https://github.com/apache/arrow-datafusion/pull/436#discussion_r642666851
##
File path: datafusion/src/optimizer/remove_duplicate_filters.rs
##
@@ -0,0 +1,611 @@
+// regarding copyright ownership. The ASF licenses this
Dandandan commented on a change in pull request #436:
URL: https://github.com/apache/arrow-datafusion/pull/436#discussion_r64299
##
File path: datafusion/src/optimizer/remove_duplicate_filters.rs
##
@@ -0,0 +1,611 @@
+// regarding copyright ownership. The ASF licenses this
alippai edited a comment on issue #451:
URL:
https://github.com/apache/arrow-datafusion/issues/451#issuecomment-851672062
In this case LSQB sounds to be a better first target. đ
> So also into what vectorized engines (can) do here.
I have a bad experience with dedicated "grap
alippai commented on issue #451:
URL:
https://github.com/apache/arrow-datafusion/issues/451#issuecomment-851672062
In this case LSQB sounds to be a better first target. đ
> So also into what vectorized engines (can) do here.
I have a bad experience with dedicated "graph engines",
Dandandan commented on a change in pull request #436:
URL: https://github.com/apache/arrow-datafusion/pull/436#discussion_r642666286
##
File path: datafusion/src/optimizer/remove_duplicate_filters.rs
##
@@ -0,0 +1,611 @@
+// regarding copyright ownership. The ASF licenses this
Dandandan opened a new issue #461:
URL: https://github.com/apache/arrow-datafusion/issues/461
*Is your feature request related to a problem or challenge? Please describe
what you are trying to do.*
We should support semi join. This can also be used to execute IN and EXISTS.
Explicit s
Dandandan opened a new issue #460:
URL: https://github.com/apache/arrow-datafusion/issues/460
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
We should support anti join. This can also be used to execute `NOT IN` and
`NOT EXISTS`
Dandandan commented on issue #451:
URL:
https://github.com/apache/arrow-datafusion/issues/451#issuecomment-851666274
Thanks a lot again đ
I think the challenging part with recursive CTE in DataFusion will be doing
it efficiently with arrow data, as .
So also into what vectorized
codecov-commenter edited a comment on pull request #382:
URL: https://github.com/apache/arrow-rs/pull/382#issuecomment-850953216
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/382?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_ter
jgoday commented on pull request #436:
URL: https://github.com/apache/arrow-datafusion/pull/436#issuecomment-851664134
@Dandandan I've just added some more simplification rules (from
https://github.com/Dandandan/datafusion-tokomak/blob/main/src/lib.rs#L44, as
you mentioned before), What do
Dandandan commented on a change in pull request #459:
URL: https://github.com/apache/arrow-datafusion/pull/459#discussion_r642658594
##
File path: ballista/rust/core/src/execution_plans/query_stage.rs
##
@@ -77,16 +109,142 @@ impl ExecutionPlan for QueryStageExec {
) -> Re
Dandandan commented on a change in pull request #459:
URL: https://github.com/apache/arrow-datafusion/pull/459#discussion_r642657693
##
File path: ballista/rust/core/src/execution_plans/query_stage.rs
##
@@ -31,26 +45,44 @@ use uuid::Uuid;
#[derive(Debug, Clone)]
pub struct Q
codecov-commenter edited a comment on pull request #459:
URL: https://github.com/apache/arrow-datafusion/pull/459#issuecomment-851638878
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/459?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+
codecov-commenter edited a comment on pull request #382:
URL: https://github.com/apache/arrow-rs/pull/382#issuecomment-850953216
#
[Codecov](https://codecov.io/gh/apache/arrow-rs/pull/382?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_ter
alippai commented on issue #451:
URL:
https://github.com/apache/arrow-datafusion/issues/451#issuecomment-851647603
For the LSQB here is the paper
https://szarnyasg.github.io/tsmb-grades21/ms.pdf and a presentation
https://docs.google.com/presentation/d/1pxyX_CWhFVYEttjTG2BrzuaMkEuLRxfhf5i
alippai commented on issue #451:
URL:
https://github.com/apache/arrow-datafusion/issues/451#issuecomment-851646571
@Dandandan I'm not sure on the recursive
CTE implementatiomm part, however PostgreSQL has a brief description on the
algorithm
https://www.postgresql.org/docs/current/que
codecov-commenter commented on pull request #459:
URL: https://github.com/apache/arrow-datafusion/pull/459#issuecomment-851638878
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/459?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comment
Dandandan commented on issue #451:
URL:
https://github.com/apache/arrow-datafusion/issues/451#issuecomment-851634623
I didn't hear of this benchmark before, thanks for referencing it! Sounds
really cool/useful.
I believe for graph processing you'll need (mostly) support for recursiv
andygrove commented on pull request #459:
URL: https://github.com/apache/arrow-datafusion/pull/459#issuecomment-851632933
@edrevo fyi
--
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
andygrove opened a new pull request #459:
URL: https://github.com/apache/arrow-datafusion/pull/459
# Which issue does this PR close?
Closes #458
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing chang
andygrove opened a new issue #458:
URL: https://github.com/apache/arrow-datafusion/issues/458
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
This issue is to track some refactoring work in preparation for implementing
map-side s
mluts opened a new pull request #457:
URL: https://github.com/apache/arrow-datafusion/pull/457
# Which issue does this PR close?
Closes #186 .
# Rationale for this change
Making it easier for newcomers to run example code.
Unfortunately i didn't find any
pachadotdev commented on a change in pull request #:
URL: https://github.com/apache/arrow/pull/#discussion_r642601213
##
File path: r/tests/testthat/test-dplyr-mutate.R
##
@@ -32,59 +51,23 @@ test_that("mutate() is lazy", {
)
})
-test_that("basic mutate", {
- exp
Dandandan edited a comment on issue #418:
URL:
https://github.com/apache/arrow-datafusion/issues/418#issuecomment-851552960
@jorgecarleitao
Interesting!
I did some earlier experiments with the vectorized hashing too (and saw
similar speed ups for low-cardinality aggregates), but
Dandandan commented on a change in pull request #68:
URL: https://github.com/apache/arrow-datafusion/pull/68#discussion_r642571782
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -339,6 +325,36 @@ pin_project! {
}
}
+fn hash_(group_values: &[ArrayRef]
Dandandan commented on a change in pull request #68:
URL: https://github.com/apache/arrow-datafusion/pull/68#discussion_r642571782
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -339,6 +325,36 @@ pin_project! {
}
}
+fn hash_(group_values: &[ArrayRef]
Dandandan commented on a change in pull request #68:
URL: https://github.com/apache/arrow-datafusion/pull/68#discussion_r642571782
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -339,6 +325,36 @@ pin_project! {
}
}
+fn hash_(group_values: &[ArrayRef]
andygrove opened a new issue #456:
URL: https://github.com/apache/arrow-datafusion/issues/456
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
As a small step towards implementing
https://github.com/apache/arrow-datafusion/issues/
Mgmaplus closed issue #10408:
URL: https://github.com/apache/arrow/issues/10408
--
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.
For queries about this service, please con
Mgmaplus commented on issue #10408:
URL: https://github.com/apache/arrow/issues/10408#issuecomment-851558916
It was just a matter of reinstalling the dependecies something in the env
was not right
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
jorgecarleitao commented on a change in pull request #68:
URL: https://github.com/apache/arrow-datafusion/pull/68#discussion_r642561933
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -339,6 +325,36 @@ pin_project! {
}
}
+fn hash_(group_values: &[Arra
Mgmaplus removed a comment on issue #10408:
URL: https://github.com/apache/arrow/issues/10408#issuecomment-851558679
It was just a matter of reinstalling the dependecies something in the env
was not right
--
This is an automated message from the Apache Git Service.
To respond to the mess
Mgmaplus commented on issue #10408:
URL: https://github.com/apache/arrow/issues/10408#issuecomment-851558679
It was just a matter of reinstalling the dependecies something in the env
was not right
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
kiszk edited a comment on pull request #10404:
URL: https://github.com/apache/arrow/pull/10404#issuecomment-851556928
Interesting. I found similar reports (e.g.
https://github.com/redis/redis/issues/6275) in other places.
It would be great to put the log of this failure into here.
kiszk commented on pull request #10404:
URL: https://github.com/apache/arrow/pull/10404#issuecomment-851556928
Interesting. I found similar reports (e.g.
https://github.com/redis/redis/issues/6275) in other places.
It would be great to put the log of this failure into here.
Is
Dandandan commented on issue #418:
URL:
https://github.com/apache/arrow-datafusion/issues/418#issuecomment-851552960
@jorgecarleitao
Interesting!
I did some earlier experiments with the vectorized hashing too (and saw
similar speed ups for low-cardinality aggregates), but got a
Dandandan commented on a change in pull request #68:
URL: https://github.com/apache/arrow-datafusion/pull/68#discussion_r642554876
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -339,6 +325,36 @@ pin_project! {
}
}
+fn hash_(group_values: &[ArrayRef]
Dandandan commented on a change in pull request #68:
URL: https://github.com/apache/arrow-datafusion/pull/68#discussion_r642554087
##
File path: datafusion/src/physical_plan/hash_aggregate.rs
##
@@ -339,6 +325,36 @@ pin_project! {
}
}
+fn hash_(group_values: &[ArrayRef]
Jimexist commented on pull request #452:
URL: https://github.com/apache/arrow-datafusion/pull/452#issuecomment-851542085
@alamb as promised :-)
--
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 th
codecov-commenter edited a comment on pull request #452:
URL: https://github.com/apache/arrow-datafusion/pull/452#issuecomment-851118576
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/452?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+
codecov-commenter commented on pull request #454:
URL: https://github.com/apache/arrow-datafusion/pull/454#issuecomment-851540857
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/454?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comment
Jimexist opened a new issue #455:
URL: https://github.com/apache/arrow-datafusion/issues/455
**Describe the bug**
A clear and concise description of what the bug is.
window function with alias is not properly rebased. select with alias will
err
**To Reproduce**
Steps to
Jimexist opened a new pull request #454:
URL: https://github.com/apache/arrow-datafusion/pull/454
# Which issue does this PR close?
fix window aggregation with alias
Closes #.
# Rationale for this change
# What changes are included in this PR?
# Ar
jorgecarleitao commented on issue #418:
URL:
https://github.com/apache/arrow-datafusion/issues/418#issuecomment-851522837
fwiw, I have used the hashing [in the experimental
branch](https://github.com/apache/arrow-datafusion/pull/68/files#diff-03876812a8bef4074e517600fdcf8e6b49f1ea24df44905
Dandandan edited a comment on issue #418:
URL:
https://github.com/apache/arrow-datafusion/issues/418#issuecomment-850491025
@ravlio
You are very right, that part is suboptimal.
Also in hash aggregates there are a couple of other things:
* Keys are created by row and indexe
Dandandan commented on issue #418:
URL:
https://github.com/apache/arrow-datafusion/issues/418#issuecomment-851518881
Thanks for the input @jhorstmann - that adds some support for the idea!
Something like ~2x speed up for more challenging queries where DF currently
"struggles" (or bigger f
nevi-me commented on a change in pull request #381:
URL: https://github.com/apache/arrow-rs/pull/381#discussion_r642516947
##
File path: parquet/src/arrow/arrow_writer.rs
##
@@ -87,17 +92,31 @@ impl ArrowWriter {
"Record batch schema does not match writer schem
github-actions[bot] commented on pull request #10411:
URL: https://github.com/apache/arrow/pull/10411#issuecomment-851476508
Revision: c40f74f0ae57f5589c139aaffc64a23ae80618c0
Submitted crossbow builds: [ursacomputing/crossbow @
actions-444](https://github.com/ursacomputing/crossbow/
kszucs commented on pull request #10411:
URL: https://github.com/apache/arrow/pull/10411#issuecomment-851475938
@github-actions crossbow submit java-jars
--
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
AlenkaF commented on a change in pull request #10334:
URL: https://github.com/apache/arrow/pull/10334#discussion_r642422637
##
File path: r/tests/testthat/test-dplyr-string-functions.R
##
@@ -493,3 +493,81 @@ test_that("edge cases in string detection and
replacement", {
t
codecov-commenter edited a comment on pull request #452:
URL: https://github.com/apache/arrow-datafusion/pull/452#issuecomment-851118576
#
[Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/452?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+
github-actions[bot] commented on pull request #10425:
URL: https://github.com/apache/arrow/pull/10425#issuecomment-851422579
https://issues.apache.org/jira/browse/ARROW-12910
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
jhorstmann commented on issue #418:
URL:
https://github.com/apache/arrow-datafusion/issues/418#issuecomment-851422139
> I think what would be bes in the long run is building a mutable typed
array based for the aggregation states, and keeping only the _offsets_ to that
array in a hash tabl
jvictorhuguenin opened a new pull request #10425:
URL: https://github.com/apache/arrow/pull/10425
âŚr month intervals
--
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.
For qu
kszucs closed pull request #10357:
URL: https://github.com/apache/arrow/pull/10357
--
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.
For queries about this service, please
Jimexist edited a comment on issue #298:
URL:
https://github.com/apache/arrow-datafusion/issues/298#issuecomment-848809874
- [x] https://github.com/apache/arrow-datafusion/pull/375 to add window
function support, streaming, and `row_number`
- [x] https://github.com/apache/arrow-datafusi
yordan-pavlov commented on a change in pull request #384:
URL: https://github.com/apache/arrow-rs/pull/384#discussion_r642402579
##
File path: parquet/src/util/memory.rs
##
@@ -292,19 +292,28 @@ impl BufferPtr {
}
/// Returns slice of data in this buffer.
+#[inl
yordan-pavlov commented on a change in pull request #384:
URL: https://github.com/apache/arrow-rs/pull/384#discussion_r642397656
##
File path: arrow/src/compute/kernels/filter.rs
##
@@ -59,19 +59,14 @@ pub(crate) struct SlicesIterator<'a> {
}
impl<'a> SlicesIterator<'a> {
-
alamb commented on pull request #383:
URL: https://github.com/apache/arrow-rs/pull/383#issuecomment-851398299
Thanks @boazberman !
Here is the Slack link @Dandandan is referring to:
https://the-asf.slack.com/archives/C01QUFS30TD/p1622403410176000
FWIW @tustvold implemented so
alamb commented on pull request #383:
URL: https://github.com/apache/arrow-rs/pull/383#issuecomment-851395966
FYI @tustvold
--
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.
alamb commented on a change in pull request #381:
URL: https://github.com/apache/arrow-rs/pull/381#discussion_r642388141
##
File path: parquet/src/arrow/arrow_writer.rs
##
@@ -1176,31 +1236,51 @@ mod tests {
let raw_values: Vec<_> = (0..SMALL_SIZE as i64).collect();
alamb commented on issue #343:
URL: https://github.com/apache/arrow-rs/issues/343#issuecomment-851388566
@nevi-me notes on #381
> we would need to account for its individual array offsets, as there is
never a guarantee that a record batch has all child arrays starting from the
sam
alamb merged pull request #450:
URL: https://github.com/apache/arrow-datafusion/pull/450
--
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.
For queries about this service, p
alamb closed issue #449:
URL: https://github.com/apache/arrow-datafusion/issues/449
--
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.
For queries about this service, please
alamb closed issue #428:
URL: https://github.com/apache/arrow-datafusion/issues/428
--
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.
For queries about this service, please
alamb merged pull request #425:
URL: https://github.com/apache/arrow-datafusion/pull/425
--
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.
For queries about this service, p
alamb closed issue #298:
URL: https://github.com/apache/arrow-datafusion/issues/298
--
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.
For queries about this service, please
alamb merged pull request #403:
URL: https://github.com/apache/arrow-datafusion/pull/403
--
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.
For queries about this service, p
alamb commented on a change in pull request #415:
URL: https://github.com/apache/arrow-datafusion/pull/415#discussion_r642379602
##
File path: .github/workflows/dev.yml
##
@@ -23,7 +23,22 @@ on:
pull_request:
jobs:
+ prettier:
+runs-on: ubuntu-latest
+steps:
+
crepererum commented on a change in pull request #381:
URL: https://github.com/apache/arrow-rs/pull/381#discussion_r642369269
##
File path: parquet/src/arrow/arrow_writer.rs
##
@@ -87,17 +92,31 @@ impl ArrowWriter {
"Record batch schema does not match writer sc
Dandandan commented on a change in pull request #384:
URL: https://github.com/apache/arrow-rs/pull/384#discussion_r642361176
##
File path: arrow/src/compute/kernels/filter.rs
##
@@ -59,19 +59,14 @@ pub(crate) struct SlicesIterator<'a> {
}
impl<'a> SlicesIterator<'a> {
-
nevi-me commented on a change in pull request #381:
URL: https://github.com/apache/arrow-rs/pull/381#discussion_r642357899
##
File path: parquet/src/arrow/arrow_writer.rs
##
@@ -87,17 +92,31 @@ impl ArrowWriter {
"Record batch schema does not match writer schem
kszucs closed pull request #10416:
URL: https://github.com/apache/arrow/pull/10416
--
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.
For queries about this service, please
kszucs commented on pull request #10416:
URL: https://github.com/apache/arrow/pull/10416#issuecomment-851341284
> > If I understand correctly this change should cancel pull request builds
as well, like subsequent pushes to the same PR.
>
> Yep
Thanks for confirming it! Let's t
potiuk commented on pull request #10416:
URL: https://github.com/apache/arrow/pull/10416#issuecomment-851320624
> If I understand correctly this change should cancel pull request builds as
well, like subsequent pushes to the same PR.
Yep
--
This is an automated message from the Ap
kszucs commented on pull request #10416:
URL: https://github.com/apache/arrow/pull/10416#issuecomment-851319673
> The configuration cancels pending jobs for the master branch, right?
If I understand correctly this change should cancel pull request builds as
well, like subsequent push
kszucs commented on pull request #10374:
URL: https://github.com/apache/arrow/pull/10374#issuecomment-851316557
The vote has passed, so closing.
--
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 t
kszucs closed pull request #10374:
URL: https://github.com/apache/arrow/pull/10374
--
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.
For queries about this service, please
Jimexist opened a new pull request #453:
URL: https://github.com/apache/arrow-datafusion/pull/453
# Which issue does this PR close?
Closes #415
# Rationale for this change
# What changes are included in this PR?
# Are there any user-facing changes?
Jimexist commented on a change in pull request #415:
URL: https://github.com/apache/arrow-datafusion/pull/415#discussion_r642280510
##
File path: .github/workflows/dev.yml
##
@@ -23,7 +23,22 @@ on:
pull_request:
jobs:
+ prettier:
+runs-on: ubuntu-latest
+steps:
crepererum commented on a change in pull request #381:
URL: https://github.com/apache/arrow-rs/pull/381#discussion_r642270850
##
File path: parquet/src/arrow/arrow_writer.rs
##
@@ -87,17 +92,31 @@ impl ArrowWriter {
"Record batch schema does not match writer sc
100 matches
Mail list logo
