8000 Use frozen::unordered_map for ErrorMessages by goedderz · Pull Request #13496 · arangodb/arangodb · GitHub
[go: up one dir, main page]
Skip to content

Use frozen::unordered_map for ErrorMessages #13496

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
Merged
merged 90 commits into from
Feb 10, 2021
Merged

Use frozen::unordered_map for ErrorMessages #13496

merged 90 commits into from
Feb 10, 2021

Conversation

goedderz
Copy link
Member
@goedderz goedderz commented Feb 5, 2021 •
edited
Loading

Scope & Purpose

As error codes and messages are known at compile time, it makes sense to save them in a compile-time map, rather than a runtime map.

  • 💩 Bugfix (requires CHANGELOG entry)
  • 🍕 New feature (requires CHANGELOG entry, feature documentation and release notes)
  • 🔥 Performance improvement
  • 🔨 Refactoring/simplification
  • 📖 CHANGELOG entry made

Backports:

  • No backports required
  • Backports required for: (Please specify versions and link PRs)

Testing & Verification

  • This change is a trivial rework / code cleanup without any test coverage.
  • The behavior in this PR was manually tested
  • This change is already covered by existing tests, such as (please describe tests).
  • This PR adds tests that were used to verify all changes:
    • Added new C++ Unit tests
    • Added new integration tests (e.g. in shell_server / shell_server_aql)
    • Added new resilience tests (only if the feature is impacted by failovers)
  • There are tests in an external testing repository:
  • I ensured this code runs with ASan / TSan or other static verification tools

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr/13868/

goedderz and others added 30 commits January 21, 2021 15:10
@goedderz
Made Result contain only an optional pointer to an error
3e30055
@goedderz
Fixed compile error
fd78e02
@goedderz
Avoid no-error allocations
df2e43c
@goedderz
WIP
d447876
@goedderz
WIP
4d8d9b1
@goedderz
Added concatT and joinT to StringUtils
ab77f45
@goedderz
Fixed tests
331fad5
@goedderz
Merge branch 'feature/string-utils-concat-join' of github.com:arangod…
bfc7901 
…b/arangodb into feature/small-result-with-external-error
@goedderz
Changed namespace from impl to detail
699a5dc
@goedderz
Assert capacity
8aa96ae
@goedderz
Fixed compile errors
b0e1b0c
@goedderz
Return a string_view in Result::errorMessage
83bb4f8
@goedderz
Removed erroneous line
c2435bc
@goedderz
Merge branch 'feature/refactor-error-message-string-view' of github.c…
6949f89 
…om:arangodb/arangodb into feature/small-result-with-external-error
@goedderz
Fixed some compile errors
0fed08d
@goedderz
Some minor changes
d91cfb5
@goedderz
Merge branch 'feature/refactor-error-message-string-view' of github.c…
6eb6ee1 
…om:arangodb/arangodb into feature/small-result-with-external-error
@goedderz
Update lib/Basics/ResultT.h
1956b21
@goedderz
Merge branch 'feature/refactor-error-message-string-view' of github.c…
f5de874 
…om:arangodb/arangodb into feature/small-result-with-external-error
@goedderz
Fixed Mac compile error
84511fb
@goedderz
Merge branch 'feature/string-utils-concat-join' of github.com:arangod…
eaacdc0 
…b/arangodb into feature/refactor-error-message-string-view
@goedderz
Merge branch 'feature/refactor-error-message-string-view' of github.c…
d988cf8 
…om:arangodb/arangodb into feature/small-result-with-external-error
@goedderz @jsteemann
Update lib/Basics/Result.h
6d83aeb 
Co-authored-by: Jan <jsteemann@users.noreply.github.com>
@goedderz
Added errorMessage move overloads
937877d
@goedderz
Added noexcept where mentioned in reviews
4d49f20
@goedderz
Be consistent.
0e098b2
@goedderz
Merge branch 'devel' of github.com:arangodb/arangodb into feature/sma…
9c209b3 
…ll-result-with-external-error
@goedderz
Removed superfluous lines
9e25112
@goedderz
Merge branch 'devel' of github.com:arangodb/arangodb into feature/sma…
bc3c0ef 
…ll-result-with-external-error
@goedderz
Fixed merge conflict
27fb946
@goedderz goedderz added this to the devel milestone Feb 5, 2021
@goedderz goedderz requested a review from jsteemann February 5, 2021 14:57
@goedderz goedderz self-assigned this Feb 5, 2021
@goedderz
Copy link
Member Author
goedderz commented Feb 5, 2021

image

@goedderz goedderz mentioned this pull request Feb 5, 2021
Merged
16 tasks
@goedderz
Copy link
Member Author
goedderz commented Feb 8, 2021

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr/13896/

@goedderz
Copy link
Member Author
goedderz commented Feb 8, 2021

image

Base automatically changed from feature/strict-typing-for-error-codes to devel February 8, 2021 16:16
@goedderz
Copy link
Member Author

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr/13927/

@goedderz
Copy link
Member Author

image

@jsteemann jsteemann merged commit eda85a9 into devel Feb 10, 2021
@jsteemann jsteemann deleted the feature/use-frozen-map-for-error-messages branch February 10, 2021 16:52
elfringham pushed a commit to elfringham/arangodb that referenced this pull request Apr 20, 2021
@goedderz @jsteemann @elfringham
Use frozen::unordered_map for ErrorMessages (arangodb#13496)
abf973c 
Co-authored-by: Jan <jsteemann@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsteemann jsteemann jsteemann approved these changes

Assignees

@goedderz goedderz

Labels
9 Review Required
Projects
None yet
Milestone
devel
Development

Successfully merging this pull request may close these issues.
2 participants
@goedderz @jsteemann
0