-
Notifications
You must be signed in to change notification settings - Fork 10
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
gpkg: bbox管理のちょっとした改善 #307
Conversation
tree_flattening: transformer::TreeFlatteningSpec::Flatten { | ||
// TODO: set properly after the flattening transformer is implemented | ||
feature: transformer::FeatureFlatteningOption::None, | ||
data: transformer::DataFlatteningOption::None, | ||
object: transformer::ObjectFlatteningOption::None, | ||
}, |
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.
TreeFlatteningはいま壊れているようなので、gpkg sinkが動くように TreeFlatteningSpec::None にしておきます。
for (table_name, bbox) in table_bboxes { | ||
tx.update_bbox(&table_name, bbox.to_tuple()).await.unwrap(); | ||
} |
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.
ふつうの into_iter に変更
@@ -195,13 +191,11 @@ impl GpkgSink { | |||
.await | |||
.unwrap(); | |||
|
|||
table_bboxes.get_mut(&table_name).unwrap().merge(&bbox); | |||
table_bboxes.entry(table_name).or_default().merge(&bbox); |
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.
必要になったテーブルのbboxだけ用意するように
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.
なるほど!
Codecov ReportAll modified and coverable lines are covered by tests ✅ Additional details and impacted files
📢 Thoughts on this report? Let us know! |
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.
レビュワーに指定されていませんでしたが、朝LTで触る際についでに見ちゃいました!
LGTM!
登場したテーブルのみのbboxを管理して、gpkg_contentsを更新するようにします。(ちょっとした変更なので他の作業とは特段コンプリクトしないと思います)。
(その他、いまの main に unused imports が色々と混入しているので、除去します)