Skip to content

Commit

Permalink
GH-35144: [C++] Fix a unit test broken when the output order of the a…
Browse files Browse the repository at this point in the history
…ggregate node changed (#35145)

### Rationale for this change

The unit test was broken by a recent change and needed to be updated.

### What changes are included in this PR?

The unit test has been updated to expect the correct order.

### Are these changes tested?

Yes

### Are there any user-facing changes?

No
* Closes: #35144

Authored-by: Weston Pace <weston.pace@gmail.com>
Signed-off-by: Weston Pace <weston.pace@gmail.com>
  • Loading branch information
westonpace committed Apr 14, 2023
1 parent 2ba6628 commit d4f7332
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cpp/src/arrow/acero/hash_aggregate_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ TEST(AggregateSchema, SingleKeyAndSegmentKey) {
aggregate::MakeOutputSchema(input_schema, {FieldRef("y")}, {FieldRef("z")},
{{"hash_count", nullptr, "x", "hash_count"}}));
AssertSchemaEqual(
schema({field("y", int32()), field("z", int32()), field("hash_count", int64())}),
schema({field("z", int32()), field("y", int32()), field("hash_count", int64())}),
output_schema);
}

Expand Down

0 comments on commit d4f7332

Please sign in to comment.