[GitHub] [arrow-datafusion] Dandandan commented on a change in pull request #365: Refactor: move RowGroupPredicateBuilder into its own module, rename to PruningPredicateBuilder

2021-05-19 Thread GitBox
Dandandan commented on a change in pull request #365: URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635557126 ## File path: datafusion/src/physical_optimizer/pruning.rs ## @@ -0,0 +1,769 @@ +// Licensed to the Apache Software Foundation (ASF) under one +/

[GitHub] [arrow-datafusion] Dandandan commented on a change in pull request #365: Refactor: move RowGroupPredicateBuilder into its own module, rename to PruningPredicateBuilder

2021-05-19 Thread GitBox
Dandandan commented on a change in pull request #365: URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635554150 ## File path: datafusion/src/physical_optimizer/pruning.rs ## @@ -0,0 +1,769 @@ +// Licensed to the Apache Software Foundation (ASF) under one +/

[GitHub] [arrow-datafusion] Dandandan commented on a change in pull request #365: Refactor: move RowGroupPredicateBuilder into its own module, rename to PruningPredicateBuilder

2021-05-19 Thread GitBox
Dandandan commented on a change in pull request #365: URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635554027 ## File path: datafusion/src/physical_optimizer/pruning.rs ## @@ -0,0 +1,769 @@ +// Licensed to the Apache Software Foundation (ASF) under one +/

[GitHub] [arrow-datafusion] Dandandan commented on a change in pull request #365: Refactor: move RowGroupPredicateBuilder into its own module, rename to PruningPredicateBuilder

2021-05-19 Thread GitBox
Dandandan commented on a change in pull request #365: URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635553323 ## File path: datafusion/src/physical_optimizer/pruning.rs ## @@ -0,0 +1,769 @@ +// Licensed to the Apache Software Foundation (ASF) under one +/

[GitHub] [arrow-datafusion] Dandandan commented on a change in pull request #365: Refactor: move RowGroupPredicateBuilder into its own module, rename to PruningPredicateBuilder

2021-05-19 Thread GitBox
Dandandan commented on a change in pull request #365: URL: https://github.com/apache/arrow-datafusion/pull/365#discussion_r635552829 ## File path: datafusion/src/physical_optimizer/pruning.rs ## @@ -0,0 +1,769 @@ +// Licensed to the Apache Software Foundation (ASF) under one +/