Skip to content
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

insert/update/upsertにblob参照リストを渡せるようにする #165

Merged
merged 3 commits into from
Jan 17, 2025

Conversation

kuron99
Copy link
Contributor

@kuron99 kuron99 commented Jan 16, 2025

https://github.com/project-tsurugi/tsurugi-issues/issues/1115 対応のためにinsert/update/upsert APIを拡張してblob_id_typeのリストを渡せるようにします。また、互換性のためにデフォルトパラメータとして追加するので、.clang-tidyでこれを禁止しているルール( fuchsia-default-arguments-declarations )を外します。

@kuron99 kuron99 requested a review from ban-nobuhiro January 16, 2025 05:46
@kuron99 kuron99 marked this pull request as draft January 16, 2025 10:12
@kuron99
Copy link
Contributor Author

kuron99 commented Jan 16, 2025

used_blobsstd::vector<blob_id_type> で渡していたのですが、sharksfin側がこのリストをポインター+個数で渡している都合で、shirakamiも同様にポインター+個数にしたいと思います。(vectorからポインター+個数への変換は簡単だが逆はコピーが発生してしまうため)
更新後、再度レビューリクエストします。

@kuron99 kuron99 marked this pull request as ready for review January 16, 2025 20:21
@kuron99
Copy link
Contributor Author

kuron99 commented Jan 16, 2025

@ban-nobuhiro ポインター+個数を使うようにAPIを変更しました。レビューお願いします。

Copy link
Contributor

@ban-nobuhiro ban-nobuhiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

良いと思います。

@kuron99 kuron99 merged commit d10c4de into master Jan 17, 2025
14 of 16 checks passed
@kuron99
Copy link
Contributor Author

kuron99 commented Jan 17, 2025

レビューありがとうございました。マージします。

@kuron99 kuron99 deleted the pass_blob_refs branch January 17, 2025 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants