Fixes for more cppcheck issues (v2 of #101)#102
Merged
liewegas merged 25 commits intoceph:masterfrom Mar 12, 2013
dalgaaf:wip-da-sca-style-4-v2
Merged
Fixes for more cppcheck issues (v2 of #101)#102liewegas merged 25 commits intoceph:masterfrom dalgaaf:wip-da-sca-style-4-v2
liewegas merged 25 commits intoceph:masterfrom
dalgaaf:wip-da-sca-style-4-v2
Conversation
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
…list Perform initialization of IntentLogNameFilter:prefix in initialization list instead of constructor body. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Prefer prefix ++operator for non-primitive types like iterators for performance reasons. Prefix ++/-- operators avoid creating a temporary copy. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
liewegas
added a commit
that referenced
this pull request
Mar 12, 2013
Fixes for more cppcheck issues (v2 of #101) Reviewed-by: Sage Weil <sage@inktank.com>
liewegas
pushed a commit
that referenced
this pull request
Dec 14, 2016
ignore errors on informational service status
ricardoasmarques
pushed a commit
to ricardoasmarques/ceph
that referenced
this pull request
Feb 19, 2018
mgr/dashboard_v2: Create links to perf counters
galsalomon66
referenced
this pull request
in galsalomon66/ceph
Sep 17, 2022
* add rapidjson header files (https://github.com/Tencent/rapidjson master); add test application * add google-test, comparing sax parser against dom parser; editorial changes * add rapidjson [https://github.com/Tencent/rapidjson.git] as submodule. the package has been used in CEPH is too old, not contain the iterative parsing that enables the chunk-by-chunk parsing * filter key-path * from-clause accepts JSON structure * adding JSON SQL statements to be accepted by the s3select-parser. (*) new variable definition for JSON, enable access for a specific key path by the projection statements and where-clause predicates (*) editorial changes. (*) add execution flow for JSON query processing. it is not cover all planned use-cases (*) the JSON parser pushes JSON-values to scratch-area upon identifying matching key-path (*) upon JSON parser is identifying a complete row it initiates s3select execution flow (*) fix for the aggregation flow (*) fix for execution flow upon end-of-input (*) bug fix for column values (*) the aggregation is not accurate, it misses the last part of the last chunk * add case insensitive to assert * fix for aggregation flow(end-of-stream). fix case insensitive * fix for language definition to accept when/case statement as an expression. * fix for error handling in the test. * fix for exception handling. * add error handling. * a fix(temporary) for the use-case where the value is very long and may cause parsing failure upon chunk "cut" that in the middle * add an efficient flow for extracting JSON values, the flow minimizes the amount of data extraction by removing duplicate JSON-variables * fix for language definition to accept when/case statement as an expression. * rebase with row_definitions_rules * fix the execution flow per JSON, "break" the loop per each row * a fix for the use-case where the from-clause is empty. * JSON-parser: adding member-data for better describing the state of each key-value * adding JSON test queries to the main g-test * fix for exact-match filter (json variables) * fix for result handling * add star operation per JSON queries. upon star-operation(select *) the engine&JSON-parser extract *all* key-full-path and value according to from-clause and where-clause(if exists). engine&parser should be aware to JSON statement syntax, and to star-operation reside in SQL statement. the result per star-operation is displayed differently, with each column on a separate line, and each resulted row ends with row-number. * error handling: upon runtime error, to handle the exception * a fix for syntax-parser (push_json_variable) * fix for json-from-clause (remove strdup) Signed-off-by: gal salomon <gal.salomon@gmail.com> Signed-off-by: Albin Antony <aantony@redhat.com> Co-authored-by: Albin Antony <aantony@redhat.com>
Moisi
pushed a commit
to Moisi/ceph
that referenced
this pull request
Jun 30, 2025
…ctly Merge in STORAGEENG/ceph from bugfix/CEPH-214-missing-element-from-listmultipartuploads-ceph-response-qa to aka_version_18.2.1 * commit 'c6f6a3979fc36eabf6ccc260182be80bc303ca40': Handle multiple common prefixes correctly
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
These commits address issues like:
v2 of pull request #101