Skip to content

Commit

Permalink
Verify MapVector schema is not null in ArrowBridgeSchemaTest.cpp
Browse files Browse the repository at this point in the history
  • Loading branch information
whutjs committed Oct 11, 2024
1 parent 27c5e1f commit edd660c
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions velox/vector/arrow/tests/ArrowBridgeSchemaTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,11 @@ class ArrowBridgeSchemaExportTest : public testing::Test {
EXPECT_STREQ("+m", schema->format);
ASSERT_EQ(schema->n_children, 1);
schema = schema->children[0];
// Map data should be a non-nullable struct type
ASSERT_EQ(schema->flags & ARROW_FLAG_NULLABLE, 0);
ASSERT_EQ(schema->n_children, 2);
// Map data key type should be a non-nullable
ASSERT_EQ(schema->children[0]->flags & ARROW_FLAG_NULLABLE, 0);
} else if (type->kind() == TypeKind::ROW) {
EXPECT_STREQ("+s", schema->format);
}
Expand Down

0 comments on commit edd660c

Please sign in to comment.