4paradigm / 4paradigm/OpenMLDB
cast to int64_t in GetAsInt32()
Open
enhancement
minor
- Dominant language
- C++
- Stars
- 1.7k
- Forks
- 331
- Avg merge
- 12d 12h
- Merged PRs (30d)
- 1
Description
https://github.com/4paradigm/OpenMLDB/blob/0c2592da4da94d34804560ada03dd743aa7f25b5/hybridse/include/node/sql_node.h#L975-L991
Contributor guide
Research direction
Look at hybridse/include/node/sql_node.h lines 975-991, specifically the GetAsInt32() method. The issue suggests adding a cast to int64_t. Check the surrounding code to understand the data types involved and ensure the cast is safe. Run any existing tests related to SQL node parsing or integer conversions to verify the change.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- databases
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100