Skip to content

Commit

Permalink
chore: Fixed tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Weijun-H committed Feb 2, 2025
1 parent cdd3197 commit 54db067
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
// specific language governing permissions and limitations
// under the License.

//! This file implements the [`RepartitionExec`] operator, which maps N input
//! This file implements the [`OnDemandRepartitionExec`] operator, which maps N input
//! partitions to M output partitions based on a partitioning scheme, optionally
//! maintaining the order of the input rows in the output.
Expand Down
9 changes: 4 additions & 5 deletions datafusion/sqllogictest/test_files/group_by.slt
Original file line number Diff line number Diff line change
Expand Up @@ -3429,11 +3429,10 @@ physical_plan
05)--------CoalesceBatchesExec: target_batch_size=4
06)----------RepartitionExec: partitioning=Hash([sn@0, amount@1], 8), input_partitions=8
07)------------AggregateExec: mode=Partial, gby=[sn@1 as sn, amount@2 as amount], aggr=[sum(l.amount)]
08)--------------ProjectionExec: expr=[amount@1 as amount, sn@2 as sn, amount@3 as amount]
09)----------------NestedLoopJoinExec: join_type=Inner, filter=sn@0 >= sn@1
10)------------------MemoryExec: partitions=1, partition_sizes=[1]
11)------------------RepartitionExec: partitioning=RoundRobinBatch(8), input_partitions=1
12)--------------------MemoryExec: partitions=1, partition_sizes=[1]
08)--------------NestedLoopJoinExec: join_type=Inner, filter=sn@0 >= sn@1, projection=[amount@1, sn@2, amount@3]
09)----------------MemoryExec: partitions=1, partition_sizes=[1]
10)----------------RepartitionExec: partitioning=RoundRobinBatch(8), input_partitions=1
11)------------------MemoryExec: partitions=1, partition_sizes=[1]

query IRR
SELECT r.sn, SUM(l.amount), r.amount
Expand Down

0 comments on commit 54db067

Please sign in to comment.