4paradigm / 4paradigm/OpenMLDB

udf test for `unix_timestamp("")` might fail

Open
#2,998 5 comments 0 reactions 2 assignees Claimed by @aceforeverd View on GitHub
bug good first issue test
Dominant language
C++
Stars
1.7k
Forks
331
Avg merge
12d 12h
Merged PRs (30d)
1

Description

```log
/home/runner/work/OpenMLDB/OpenMLDB/hybridse/src/udf/udf_test.h:172: Failure
Expected equality of these values:
expect
Which is: Nullable{value=1674015278, type=int64}
output
Which is: Nullable{value=1674015279, type=int64}
```

refer https://github.com/4paradigm/OpenMLDB/actions/runs/3945482097/jobs/6752411020#step:7:21561

Fail might happen due to slow execution.

Contributor guide

Open the contributing guide

Research direction

The failing test is in hybridse/src/udf/udf_test.h at line 172, testing the unix_timestamp function with an empty string. The test expects a specific timestamp but gets a slightly different one, likely due to timing. Start by examining the test logic and the UDF implementation for unix_timestamp. Run the test locally to reproduce the failure, then consider mocking time or adjusting the test tolerance. 'Done' means the test passes consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
testing
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
40/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.