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.

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

Fixed web service crash #4334

Merged
merged 2 commits into from
Jun 20, 2022
Merged

Fixed web service crash #4334

merged 2 commits into from
Jun 20, 2022

Conversation

dutor
Copy link
Contributor

@dutor dutor commented Jun 20, 2022

Check if folly::Optional has value first.

@dutor dutor added the ready-for-testing PR: ready for the CI test label Jun 20, 2022
@Sophie-Xie Sophie-Xie added the cherry-pick-v3.2 PR: need cherry-pick to this version label Jun 20, 2022
@dutor dutor merged commit 533d359 into vesoft-inc:master Jun 20, 2022
@dutor dutor deleted the fix-web branch June 20, 2022 05:37
nebula-bots pushed a commit that referenced this pull request Jun 23, 2022
Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>
Sophie-Xie added a commit that referenced this pull request Jun 27, 2022
Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>
Sophie-Xie added a commit that referenced this pull request Jun 27, 2022
* force cache the docker layer (#4331)

* check god role exist when meta init (#4330)

* check god role exist when meta init

* return error if kv fail

Co-authored-by: Doodle <13706157+critical27@users.noreply.github.com>

* Fix object pool mtsafe. (#4332)

* Fix object pool mtsafe.

* Fix lock.

* Fixed web service crash (#4334)

Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>

* Fix get edges transform rule. (#4328)

1. Input/Ouput variables.
2. Keep column names of Limit same with input plan node.

Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>

* fix rc docker (#4336)

* add lock (#4352)

* fix map concurrency issue (#4344)

* fix mutex in map

* add test

* move the order

Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>

* add stats under index conditions (#4353)

Co-authored-by: Harris.Chu <1726587+HarrisChu@users.noreply.github.com>
Co-authored-by: jimingquan <mingquan.ji@vesoft.com>
Co-authored-by: Doodle <13706157+critical27@users.noreply.github.com>
Co-authored-by: shylock <33566796+Shylock-Hg@users.noreply.github.com>
Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Co-authored-by: panda-sheep <59197347+panda-sheep@users.noreply.github.com>
@jinyingsunny jinyingsunny added the severity/major Severity of bug label Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick-v3.2 PR: need cherry-pick to this version ready-for-testing PR: ready for the CI test severity/major Severity of bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants