Skip to content
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

Jump to bottom

minimal value of i64 can not be expressed in tremor script #127

Closed
Licenser opened this issue Mar 18, 2020 · 1 comment · Fixed by #629
Closed

minimal value of i64 can not be expressed in tremor script #127

Licenser opened this issue Mar 18, 2020 · 1 comment · Fixed by #629
Labels
bug Something isn't working

Comments

@Licenser
Copy link
Member

Problem
The literal -9223372036854775808 can't be represented by a script

Steps

Run the following script:

-9223372036854775808

Notes

Output of tremor-server --version: 0.7.3
@Licenser Licenser added the bug Something isn't working label Mar 18, 2020
@Licenser
Copy link
Member Author

renaming for clarity

@Licenser Licenser changed the title min (integer) can not be written in a script minimal value of i64 can not be expressed in tremor script Apr 16, 2020
m0n0chr0m3 pushed a commit to m0n0chr0m3/tremor-runtime that referenced this issue Apr 16, 2020
@trenaux
Proof-of-concept fix for issue tremor-rs#127.
58a9065 
May need to be extended to deal with leading decimal point,
octal/hexadecimal numbers, etc. (and perhaps the parser grammar can be
even be simplified).
@mfelsche mfelsche mentioned this issue Dec 2, 2020
Merged
5 tasks
Licenser added a commit that referenced this issue Mar 4, 2022
@Licenser
Fix #127 integer parsing
f41cc09 
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
Licenser added a commit that referenced this issue Mar 9, 2022
@Licenser
Fix #127 integer parsing
4a3bbed 
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
Licenser added a commit that referenced this issue Mar 9, 2022
@Licenser
Fix #127 integer parsing
6a1adff 
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
Licenser added a commit that referenced this issue Mar 10, 2022
@Licenser
Fix #127 integer parsing
bab8a96 
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
mfelsche pushed a commit that referenced this issue Mar 21, 2022
@Licenser @mfelsche
Fix #127 integer parsing
e0e6e2e 
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
mfelsche pushed a commit that referenced this issue Mar 22, 2022
@Licenser @mfelsche
Fix #127 integer parsing
dd1149a 
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
Licenser added a commit that referenced this issue Apr 12, 2022
@Licenser
Fix #127 integer parsing
ff3a4d4 
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
Licenser added a commit that referenced this issue Apr 12, 2022
@Licenser
Fix #127 integer parsing
63af26c 
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Output more helpful error messages for pipeline runtime errors tremor-rs/tremor-runtime
1 participant
@Licenser