-
Notifications
You must be signed in to change notification settings - Fork 240
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
Add support for old db3 schema used on distros prior to Foxy #1090
Add support for old db3 schema used on distros prior to Foxy #1090
Conversation
Signed-off-by: Michael Orlov <michael.orlov@apex.ai>
- Added unit tests for db with old schema - Added unit test for new `database.is_field_exist()` API Signed-off-by: Michael Orlov <michael.orlov@apex.ai>
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.
Looks really well done! The only thing I'd change is a minor rename to have is_field_exist
look more similar to other exists
functions. But that's minor enough that I'm not comfortable holding things up for it.
...g2_storage_default_plugins/include/rosbag2_storage_default_plugins/sqlite/sqlite_wrapper.hpp
Outdated
Show resolved
Hide resolved
rosbag2_storage_default_plugins/src/rosbag2_storage_default_plugins/sqlite/sqlite_storage.cpp
Outdated
Show resolved
Hide resolved
rosbag2_storage_default_plugins/src/rosbag2_storage_default_plugins/sqlite/sqlite_wrapper.cpp
Outdated
Show resolved
Hide resolved
Signed-off-by: Michael Orlov <michael.orlov@apex.ai>
Gist: https://gist.githubusercontent.com/MichaelOrlov/ab796029b394e54ca6b2fa2a56189374/raw/340d151e32db0a4912eee351dbf7fe86518ca0bb/ros2.repos |
https://github.com/Mergifyio backport humble |
✅ Backports have been created
|
* Support `SqliteStorage::get_metadata()` for files from `eloquent` Signed-off-by: Michael Orlov <michael.orlov@apex.ai> * Add unit tests - Added unit tests for db with old schema - Added unit test for new `database.is_field_exist()` API Signed-off-by: Michael Orlov <michael.orlov@apex.ai> * Rename `is_filed_exist(..)` to `field_exists(..)` Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Signed-off-by: Michael Orlov <michael.orlov@apex.ai> (cherry picked from commit c5761ee)
…1094) * Support `SqliteStorage::get_metadata()` for files from `eloquent` Signed-off-by: Michael Orlov <michael.orlov@apex.ai> * Add unit tests - Added unit tests for db with old schema - Added unit test for new `database.is_field_exist()` API Signed-off-by: Michael Orlov <michael.orlov@apex.ai> * Rename `is_filed_exist(..)` to `field_exists(..)` Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Signed-off-by: Michael Orlov <michael.orlov@apex.ai> (cherry picked from commit c5761ee) Co-authored-by: Michael Orlov <michael.orlov@apex.ai>
https://github.com/Mergifyio backport galactic |
✅ Backports have been created
|
* Support `SqliteStorage::get_metadata()` for files from `eloquent` Signed-off-by: Michael Orlov <michael.orlov@apex.ai> * Add unit tests - Added unit tests for db with old schema - Added unit test for new `database.is_field_exist()` API Signed-off-by: Michael Orlov <michael.orlov@apex.ai> * Rename `is_filed_exist(..)` to `field_exists(..)` Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Signed-off-by: Michael Orlov <michael.orlov@apex.ai> (cherry picked from commit c5761ee)
…1105) * Support `SqliteStorage::get_metadata()` for files from `eloquent` Signed-off-by: Michael Orlov <michael.orlov@apex.ai> * Add unit tests - Added unit tests for db with old schema - Added unit test for new `database.is_field_exist()` API Signed-off-by: Michael Orlov <michael.orlov@apex.ai> * Rename `is_filed_exist(..)` to `field_exists(..)` Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Signed-off-by: Michael Orlov <michael.orlov@apex.ai> (cherry picked from commit c5761ee) Co-authored-by: Michael Orlov <michael.orlov@apex.ai>
This pull request has been mentioned on ROS Discourse. There might be relevant details there: https://discourse.ros.org/t/ros-2-tsc-meeting-minutes-2022-10-13/28213/1 |
info
andplay
are not able to open db3 bag files recorded on eloquent #1089Support SqliteStorage::get_metadata() for db3 files recorded on distros prior to the
Foxy
by checking whetheroffered_qos_profiles
field exist or not intopics
table and forming SQL request accordingly.