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.

Sign up for GitHub

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

Jump to bottom

Autoincrement index #709

Merged
merged 171 commits into from
Oct 22, 2020
Merged

Autoincrement index #709

merged 171 commits into from
Oct 22, 2020

Conversation

kulmann
Copy link
Contributor

@kulmann kulmann commented Oct 15, 2020 •
edited
Loading

IljaN and others added 30 commits October 14, 2020 18:03
@IljaN
Storage Index [WIP]
85743a2
@IljaN
WIP
e054bfc
@IljaN
WIP Reflection type-names
6f05c09
@IljaN
Rename index package to indexer
36f8931
@IljaN
FindBy + tests
a8f7d5d
@IljaN
Delete + tests
ded569f
@IljaN
FindByPartial + tests
930f446
@IljaN
Add Update + tests to indexer
2a77e2b
@IljaN
Restructure in to subpackages
36c4d15
@IljaN
dot import alias for test pkg
cfafdb6
@IljaN
add reflect_test
1c3e47c
@IljaN
Fix errors alias name
683138f
@IljaN
Create pkg for index disk implementation
e70fbea
@IljaN
WIP cs3 index
f9ee733
@IljaN
cs3 unique index implementation
66900b4
@refs @IljaN
fix indexer folder
d2d8650
@refs @IljaN
non_unique add impl
0ddd3d7
@refs @IljaN
non_unique cs3 index: final implementation
5be1cb1
@refs @IljaN
fix index stringify name
fed18db
@refs @IljaN
fix linter
6373569
@IljaN
Extract data-provider base url to the respective client
60cc905
@refs @IljaN
re-implement update
e32738e
@refs @IljaN
wip config
a852edc
@IljaN
Apply ProviderAddr from options to cs3config
4300f6a
@IljaN
Add non-unique with index with options
dba08c4
@refs @IljaN
merge AddUniqueIndex and AddNonUniqueIndex into AddIndex to the Indexer
fbd52ce
@refs @IljaN
delete unused AddNonUniqueIndex
4444f5e
@refs @IljaN
fix test suite
4015a50
@kulmann @IljaN
start metadata storage in tests
7bbc387
@refs @IljaN
refactor disk tests
04b3371
@refs refs changed the base branch from filesystem_index to master October 22, 2020 14:39
@refs refs marked this pull request as ready for review October 22, 2020 14:44
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@kulmann kulmann requested a review from refs October 22, 2020 16:09
@kulmann
Copy link
Contributor Author

kulmann commented Oct 22, 2020

This PR had a successful CI run, except from a known random failure because of a phoenix logout issue in CI, here: https://drone.owncloud.com/owncloud/ocis/888

@micbar micbar merged commit 5a3c0e5 into master Oct 22, 2020
@delete-merged-branch delete-merged-branch bot deleted the autoincrement-index branch October 22, 2020 16:51
This was referenced Oct 22, 2020
Closed
Closed
ownclouders pushed a commit that referenced this pull request Oct 22, 2020
@micbar
commit 5a3c0e5
09376c9 
Merge: fe96475 29b6930
Author: Michael Barz <[email protected]>
Date:   Thu Oct 22 18:51:09 2020 +0200

    Merge pull request #709 from owncloud/autoincrement-index

    Autoincrement index
@IljaN IljaN mentioned this pull request Oct 23, 2020
Closed
This was referenced Oct 27, 2020
Closed
Closed
@micbar micbar mentioned this pull request Dec 16, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@refs refs refs approved these changes

Assignees

@kulmann kulmann

@refs refs

Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@kulmann @refs @micbar @IljaN