Skip to content

fix: Flaky test FunctionMetadataTest.mod #26441

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aditi-pandit merged 7 commits into from
Nov 20, 2025
+23 −27
Merged

fix: Flaky test FunctionMetadataTest.mod #26441

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
2f4af55
Fix failure of FunctionMetadataTest.mod on MacOS
rkurniawati Oct 26, 2025
6ee41d7
Fix code formatting
rkurniawati Oct 27, 2025
ad45824
Consolidate sorting of constraint arrays in metadata and metadata ele…
rkurniawati Oct 27, 2025
f6c7646
Use folly::hasher in the sortMetadataList, change sortMetadataList pa…
rkurniawati Oct 28, 2025
8586eb4
Fix test failure due to an extraneous metadata
rkurniawati Oct 28, 2025
50a1ebb
Sort the function metadata by the function kind and parameter types
rkurniawati Nov 4, 2025
8c705a2
Use basic_json::dump to convert json::array_t to string, remove unnec…
rkurniawati Nov 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions presto-native-execution/presto_cpp/main/functions/tests/FunctionMetadataTest.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,16 @@ class FunctionMetadataTest : public ::testing::Test {
functionMetadata_ = getFunctionsMetadata();
}

static void sortConstraintArrays(json& metadata) {
for (auto const& [key, val] : metadata.items()) {
if (key.ends_with("Constraints") && metadata[key].is_array()) {
std::sort(metadata[key].begin(), metadata[key].end(), [](const json& a, const json& b) {
return a.dump() < b.dump();
});
}
}
}

void testFunction(
const std::string& name,
const std::string& expectedFile,
Expand All @@ -59,13 +69,20 @@ class FunctionMetadataTest : public ::testing::Test {
std::sort(expectedList.begin(), expectedList.end(), comparator);
std::sort(metadataList.begin(), metadataList.end(), comparator);
for (auto i = 0; i < expectedSize; i++) {
// Constraint arrays are coming from unordered map, they need to be sorted so that
// differences in the element order will not cause test failure.

sortConstraintArrays(expectedList[i]);
sortConstraintArrays(metadataList[i]);

EXPECT_EQ(expectedList[i], metadataList[i]) << "Position: " << i;
}
}

json functionMetadata_;
};


TEST_F(FunctionMetadataTest, approxMostFrequent) {
testFunction("approx_most_frequent", "ApproxMostFrequent.json", 7);
}
Expand Down
Loading