-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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.
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
[0.10.0_nightly_308cab0-1] panic: runtime error: index out of range #5202
Comments
@ivanscattergood can you give some context as to what was happening? Was this during startup? Did this happen during normal load? Does it recover on restart? |
Hi, Sorry, I realise now in isolation that error message is useless! I have been running the 0.9.6.1 binary for just about a week and saving collecting 0.5 million datapoints (over 1 million series) every 10 minutes. I wanted to upgrade to the nightly after advice from @pauldix (in ticket #4977). When I upgraded to the new binary, this was the error I got on startup. I have rolled back to 0.9.6.1 and it works fine. Will I need to start with a Fresh database to use the nightly? |
@jwilder -- any thoughts? |
If you have existing |
Please wipe your tsm database and try again. |
I think there's some confusion here. There shouldn't be any breaking changes between the 0.9.6.1 TSM format and master. Would you be able to send us the TSM file that it's crashing on? |
@ivanscattergood Also, could you attach the output of an |
The raw data is getting saved in a shard called detail: [root@visual-edge detail]# pwd [root@visual-edge detail]# ls -ltrah And the offending directory 16: Which TSM file would you like to see? |
I cannot load the TSM files via this interface as the largest upload size is 10mb, is there another way I can send you these files? |
The block count was an uint16 when incrementing the index location which was an int32. This caused the value the uint16 value to overflow before the index location was incremented causing the wrong location to be read on the next iteration of the loop. This triggers the slice out of range errors. Added a test that recreates the panic seen in #5257 and possibly #5202 which is older code. Fixes #5257
@ivanscattergood If you are still seeing this issue, #5264 may have fixed it. That fix should be in tonights nightly build. |
Fixed via #5264 |
@jwilder close then? |
I'm Happy to close this issue |
Thanks! Forgot to close. |
The text was updated successfully, but these errors were encountered: