Skip to content

Commit

Permalink
Support TDigestType in Presto to Velox Query Plan (#24546)
Browse files Browse the repository at this point in the history
Summary:
X-link: facebookincubator/velox#12326


Differential Revision: D69558489
  • Loading branch information
natashasehgal authored and facebook-github-bot committed Feb 13, 2025
1 parent d08c27e commit 5350cbf
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
#include "velox/functions/prestosql/types/IPAddressType.h"
#include "velox/functions/prestosql/types/IPPrefixType.h"
#include "velox/functions/prestosql/types/JsonType.h"
#include "velox/functions/prestosql/types/TDigestType.h"
#include "velox/functions/prestosql/types/TimestampWithTimeZoneType.h"
#include "velox/functions/prestosql/types/UuidType.h"

Expand Down Expand Up @@ -50,6 +51,7 @@ class PrestoToVeloxQueryPlanTest : public ::testing::Test {
registerIPAddressType();
registerIPPrefixType();
registerJsonType();
registerTDigestType();
registerTimestampWithTimeZoneType();
registerUuidType();
}
Expand Down Expand Up @@ -111,7 +113,7 @@ TEST_F(PrestoToVeloxQueryPlanTest, parseSqlFunctionHandleAllComplexTypes) {
std::string str = R"(
{
"@type": "json_file",
"functionId": "json_file.test.all_complex_types;row(map(hugeint, ipaddress), ipprefix);row(array(varbinary), timestamp, date, json, hyperloglog, timestamp with time zone, interval year to month, interval day to second);function(double, boolean);uuid",
"functionId": "json_file.test.all_complex_types;row(map(hugeint, ipaddress), ipprefix);row(array(varbinary), timestamp, date, json, hyperloglog, tdigest, timestamp with time zone, interval year to month, interval day to second);function(double, boolean);uuid",
"version": "1"
}
)";
Expand All @@ -129,6 +131,7 @@ TEST_F(PrestoToVeloxQueryPlanTest, parseSqlFunctionHandleAllComplexTypes) {
DATE(),
JSON(),
HYPERLOGLOG(),
TDIGEST(),
TIMESTAMP_WITH_TIME_ZONE(),
INTERVAL_YEAR_MONTH(),
INTERVAL_DAY_TIME()}),
Expand Down

0 comments on commit 5350cbf

Please sign in to comment.