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

object/filters: Add filtering by first object ID #575

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

carpawell
Copy link
Member

This field appeared for V2 split first and was not required at first glance. More use cases have been landed now. Also, it is a successor to the SplitID in the V1 split that does have a filter.

API go repo is accepted for archival so no additional const importing for now, will be fixed somehow with removing neofs-api-go from go.mod.

object/search.go Outdated Show resolved Hide resolved
object/search_test.go Outdated Show resolved Hide resolved
@carpawell carpawell force-pushed the fix/add-v2-split-filters branch from d6eae01 to 8e7583a Compare April 11, 2024 07:51
object/search.go Outdated Show resolved Hide resolved
@carpawell carpawell force-pushed the fix/add-v2-split-filters branch from 8e7583a to 9a67931 Compare April 11, 2024 07:58
object/search_test.go Outdated Show resolved Hide resolved
This field appeared for V2 split first and was not required at first glance.
More use cases have been landed now. Also, it is a successor to the SplitID in
the V1 split that does have a filter.

api-go repo is accepted for archival so no additional const importing for now,
will be fixed somehow with removing `neofs-api-go` from go.mod.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
@carpawell carpawell force-pushed the fix/add-v2-split-filters branch from 9a67931 to 10c25ff Compare April 11, 2024 12:06
@carpawell carpawell requested a review from cthulhu-rider April 11, 2024 12:06
@cthulhu-rider cthulhu-rider merged commit acc1338 into master Apr 11, 2024
7 of 10 checks passed
@cthulhu-rider cthulhu-rider deleted the fix/add-v2-split-filters branch April 11, 2024 12:50
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.

3 participants