Skip to content

Commit 8c37a14

Browse files
committed
Add support for setting sessionStartTime in VeloxQueryConfig.
1 parent 15062a8 commit 8c37a14

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

presto-native-execution/presto_cpp/main/tests/PrestoToVeloxQueryConfigTest.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -539,32 +539,32 @@ TEST_F(PrestoToVeloxQueryConfigTest, sessionStartTimeConfiguration) {
539539

540540
// Test with session start time set in SessionRepresentation
541541
// The startTime is already set in createBasicSession() to 1234567890
542-
auto veloxConfig = toVeloxConfigs(session);
542+
auto veloxConfig = QueryConfig{toVeloxConfigs(session)};
543543

544544
// Verify that session start time is properly passed through to VeloxQueryConfig
545545
EXPECT_EQ(1234567890, veloxConfig.sessionStartTimeMs());
546546

547547
// Test with different session start time
548548
session.startTime = 9876543210;
549-
auto veloxConfig2 = toVeloxConfigs(session);
549+
auto veloxConfig2 = QueryConfig{toVeloxConfigs(session)};
550550

551551
EXPECT_EQ(9876543210, veloxConfig2.sessionStartTimeMs());
552552

553553
// Test with zero start time (valid edge case)
554554
session.startTime = 0;
555-
auto veloxConfig3 = toVeloxConfigs(session);
555+
auto veloxConfig3 = QueryConfig{toVeloxConfigs(session)};
556556

557557
EXPECT_EQ(0, veloxConfig3.sessionStartTimeMs());
558558

559559
// Test with negative start time (valid edge case)
560560
session.startTime = -1000;
561-
auto veloxConfig4 = toVeloxConfigs(session);
561+
auto veloxConfig4 = QueryConfig{toVeloxConfigs(session)};
562562

563563
EXPECT_EQ(-1000, veloxConfig4.sessionStartTimeMs());
564564

565565
// Test with maximum value
566566
session.startTime = std::numeric_limits<int64_t>::max();
567-
auto veloxConfig5 = toVeloxConfigs(session);
567+
auto veloxConfig5 = QueryConfig{toVeloxConfigs(session)};
568568

569569
EXPECT_EQ(
570570
std::numeric_limits<int64_t>::max(), veloxConfig5.sessionStartTimeMs());

0 commit comments

Comments
 (0)