나는 아무것도 생각하지 않지만 재귀 적으로 작동하면 좋을 것이라고 믿습니다.

rcutils_calculate_directory_size 는 현재 rosbag2 메타데이터 를 계산하는 데 사용됩니다. 디렉토리의 모든 파일이 있으므로 지금 당장은 재귀적일 필요가 없습니다.

rcutils_calculate_directory_size는 현재 rosbag2 메타데이터를 계산하는 데 사용됩니다. 디렉토리의 모든 파일이 있으므로 지금 당장은 재귀적일 필요가 없습니다.

예. 이제 rosbag2의 metadata_io.cpp 이 함수를 호출합니다.

재귀 검색을 지원하지 않는다고 명시적으로 설명합니다. 실제 사용 사례에서 재귀적이어야 합니까?

재귀 검색을 지원하지 않는다고 명시적으로 설명합니다. 실제 사용 사례에서 재귀적이어야 합니까?

예. 아래와 같이 rosbag2의 현재 구현

https://github.com/ros2/rosbag2/blob/700817c42f1bbc26ced79aca2186094350c8d1be/rosbag2_storage/src/rosbag2_storage/metadata_io.cpp#L229 -L240

지정된 경로의 총 파일 크기만 계산합니다. sqlite3 데이터베이스의 경우(이제 rosbag2는 이것을 스토리지 백엔드로만 제공합니다.) sqlite3 데이터베이스는 하나의 파일이기 때문에 괜찮습니다. 그러나 다른 사람들(예: leveldb)의 경우 디렉토리를 사용하여 데이터를 저장합니다. 위의 코드는 데이터베이스의 정확한 크기를 얻을 수 없습니다.
재귀를 지원하도록 이 함수를 업데이트하고 싶습니다.
이 수정으로 인해 현재 코드에 문제가 발생해서는 안 됩니다.

재귀를 지원하도록 이 함수를 업데이트하고 싶습니다.
이 수정으로 인해 현재 코드에 문제가 발생해서는 안 됩니다.

예, 그렇게 하기 위해 그것을 향상시키는 데 문제가 없다고 봅니다. 호출자가 실제로 원하지 않는 경우를 대비하여 함수에 recursive 부울 매개변수를 도입할 수 있지만 그렇지 않으면 나에게는 좋게 들립니다.

@clalancette
CC: @Barry-Xu-2018

댓글 감사합니다!
실제로 우리는 leveldb 를 rosbag2 백엔드 데이터베이스로 사용하려고 했습니다. 이를 위해 이 개선 사항이 필요합니다.

@clalancette @fujitatomoya

호출자가 실제로 원하지 않는 경우를 대비하여 함수에 재귀 부울 매개변수를 도입하고 싶을 수도 있지만 그렇지 않으면 나에게는 좋게 들립니다.

새 매개변수를 추가하면 rcutils가 업데이트된 후 rosbag2도 동시에 업데이트해야 합니다. 그렇지 않으면 rosbag2 빌드가 실패했습니다.

Linux 플랫폼에서 테스트했습니다. 그러나 Windows 플랫폼에서 테스트해야 합니다.
이제 윈도우 환경을 준비하고 있습니다.

새 매개변수를 추가하면 rcutils가 업데이트된 후 rosbag2도 동시에 업데이트해야 합니다. 그렇지 않으면 rosbag2 빌드가 실패했습니다.

나도 이 접근 방식을 좋아하지 않는다. 인터페이스를 유지하고 재귀적으로 검색하는 기능을 향상시킬 수 있습니다. 재귀 검색이 아닌 특정 사용 사례가 있는 경우 재귀 사례에 대한 새 기능을 추가할 수 있습니다.

나도 이 접근 방식을 좋아하지 않는다. 인터페이스를 유지하고 재귀적으로 검색하는 기능을 향상시킬 수 있습니다. 재귀 검색이 아닌 특정 사용 사례가 있는 경우 재귀 사례에 대한 새 기능을 추가할 수 있습니다.

응. 동의하다.

@clalancette

새 매개변수 없이 rcutils_calculate_directory_size를 향상시키는 것은 어떻습니까?
현재 rosbag2는 향상된 rcutils_calculate_directory_size()와 함께 올바르게 작동할 수 있습니다.

새 매개변수 없이 rcutils_calculate_directory_size를 향상시키는 것은 어떻습니까?
현재 rosbag2는 향상된 rcutils_calculate_directory_size()와 함께 올바르게 작동할 수 있습니다.

내가 이것을 걱정하는 이유는 rosbag2 디렉토리 내부에 "추가" 정보를 저장하는 사용자들 사이에서 약간의 이야기가 있기 때문입니다. 하위 디렉토리에 추가 데이터를 많이 저장하게 되면 rosbag2가 기대하는 크기가 줄어들 수 있습니다.

API를 중단하고 싶지 않다면 rcutils_calculate_directory_size_recursive() 인 다른 함수를 추가할 수도 있습니다.

rcutils_calculate_directory_size() 에서 항상 재귀 검색을 수행하면 rosbag2에 문제가 발생하는지 여부에 대한 통찰력을 얻으려면 @Karsten1987 에 ping을 보내십시오.

@clalancette

나는 당신의 걱정을 이해합니다.
코드를 확인했습니다. rosbag2의 아래 코드에서만 이 기능을 사용합니다.

https://github.com/ros2/rosbag2/blob/700817c42f1bbc26ced79aca2186094350c8d1be/rosbag2_storage/src/rosbag2_storage/metadata_io.cpp#L229 -L240

코드를 변경한 후 기본 sqlite3 스토리지 플러그인으로 rosbag2도 시도했지만 문제를 찾지 못했습니다.
물론 @Karsten1987의 의견을 들어야 합니다.

@Barry-Xu-2018

나는 당신이 이것을 닫을 수 있다고 생각합니다.

306이 통합되었습니다. 따라서 이 문제를 닫습니다.