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

fix runtime error: index out of range for !amd64 env #287

Merged
merged 1 commit into from
Oct 22, 2021
Merged

fix runtime error: index out of range for !amd64 env #287

merged 1 commit into from
Oct 22, 2021

Conversation

awsl-dbq
Copy link
Contributor

@awsl-dbq awsl-dbq commented Oct 19, 2021 •
edited by manishrjain

fix runtime error: index out of range in non-amd64 (arm64) env

This change is Reviewable

Update search.go
12e6cc9 
fix `runtime error: index out of range`  in non-amd64 (arm64) env
@CLAassistant
Copy link

CLAassistant commented Oct 19, 2021 •
edited

CLA assistant check
All committers have signed the CLA.

@awsl-dbq
Copy link
Contributor Author

if Len(xs) mod 8 !=0 , then xs[i+2] may got index out of range error

NamanJain8
NamanJain8 approved these changes Oct 22, 2021
View reviewed changes
Copy link
Contributor

@NamanJain8 NamanJain8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @awsl-dbq , thanks for the fix. 🎉

@NamanJain8 NamanJain8 merged commit efb105d into dgraph-io:master Oct 22, 2021
joshua-goldstein pushed a commit that referenced this pull request Sep 15, 2022
@joshua-goldstein
fix(z): runtime error: index out of range for !amd64 env #287
4fcbae6 
fix `runtime error: index out of range`  in non-amd64 (arm64) env

(cherry picked from commit efb105d)
@joshua-goldstein joshua-goldstein mentioned this pull request Sep 15, 2022
Closed
joshua-goldstein pushed a commit that referenced this pull request Oct 10, 2022
@joshua-goldstein
fix(z): runtime error: index out of range for !amd64 env #287
016b6d5 
fix `runtime error: index out of range`  in non-amd64 (arm64) env
@joshua-goldstein joshua-goldstein mentioned this pull request Oct 10, 2022
Merged
joshua-goldstein added a commit that referenced this pull request Oct 10, 2022
@joshua-goldstein
fix(z): runtime error: index out of range for !amd64 env #287 (#307)
9b20e9a 
This PR cherry picks [PR
287](#287) into main.

Co-authored-by: brain.duan <brain.duan@ebistrategy.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NamanJain8 NamanJain8 NamanJain8 approved these changes

@manishrjain manishrjain Awaiting requested review from manishrjain

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@awsl-dbq @CLAassistant @NamanJain8