-
Notifications
You must be signed in to change notification settings - Fork 248
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
[jazzy] Add topics with zero message counts to the SQLiteStorage::get_metadata(). (backport #1725) #1731
Conversation
…a(). (#1725) * storage_sqlite3 constructs metadata with topics don't have any messages. Signed-off-by: Tomoya Fujita <[email protected]> * Revert "storage_sqlite3 constructs metadata with topics don't have any messages." This reverts commit a842689. Signed-off-by: Tomoya Fujita <[email protected]> * update metadata list topics that does not have any messages. Signed-off-by: Tomoya Fujita <[email protected]> * add get_metadata_include_topics_with_zero_messages test Signed-off-by: Tomoya Fujita <[email protected]> Co-authored-by: Michael Orlov <[email protected]> * add std_msgs to rosbag2_cpp for test build. Signed-off-by: Tomoya Fujita <[email protected]> * call get_or_generate_extern_topic_id when updating metadata. Signed-off-by: Tomoya Fujita <[email protected]> --------- Signed-off-by: Tomoya Fujita <[email protected]> Co-authored-by: Michael Orlov <[email protected]> (cherry picked from commit ff829e8)
Pulls: #1731 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. @MichaelOrlov thanks !
https://github.com/Mergifyio backport iron |
✅ Backports have been created
|
…a(). (#1725) (#1731) * storage_sqlite3 constructs metadata with topics don't have any messages. Signed-off-by: Tomoya Fujita <[email protected]> * Revert "storage_sqlite3 constructs metadata with topics don't have any messages." This reverts commit a842689. Signed-off-by: Tomoya Fujita <[email protected]> * update metadata list topics that does not have any messages. Signed-off-by: Tomoya Fujita <[email protected]> * add get_metadata_include_topics_with_zero_messages test Signed-off-by: Tomoya Fujita <[email protected]> Co-authored-by: Michael Orlov <[email protected]> * add std_msgs to rosbag2_cpp for test build. Signed-off-by: Tomoya Fujita <[email protected]> * call get_or_generate_extern_topic_id when updating metadata. Signed-off-by: Tomoya Fujita <[email protected]> --------- Signed-off-by: Tomoya Fujita <[email protected]> Co-authored-by: Michael Orlov <[email protected]> (cherry picked from commit ff829e8) Co-authored-by: Tomoya Fujita <[email protected]> (cherry picked from commit 2a627c9) # Conflicts: # rosbag2_cpp/CMakeLists.txt # rosbag2_storage_sqlite3/src/rosbag2_storage_sqlite3/sqlite_storage.cpp
rolling
.I guess we can backport this to
jazzy
.This is an automatic backport of pull request Add topics with zero message counts to the SQLiteStorage::get_metadata(). #1725 done by Mergify.