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

945 sstable interface #969

Merged
merged 31 commits into from
Nov 29, 2020
Merged

945 sstable interface #969

merged 31 commits into from
Nov 29, 2020

Conversation

itaiad200
Copy link
Contributor

No description provided.

itaiad200 and others added 30 commits November 22, 2020 16:14
Co-authored-by: itaiad200 <itaiad200@gmail.com>
…e/lakeFS into feature/rocks-catalog-interface
@itaiad200 itaiad200 requested review from ozkatz and tzahij November 29, 2020 08:55
@codecov-io
Copy link

codecov-io commented Nov 29, 2020

Codecov Report

Merging #969 (ab24d5b) into master (06806c8) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #969      +/-   ##
==========================================
- Coverage   44.58%   44.56%   -0.02%     
==========================================
  Files         142      142              
  Lines       11547    11547              
==========================================
- Hits         5148     5146       -2     
- Misses       5757     5758       +1     
- Partials      642      643       +1     
Impacted Files Coverage Δ
catalog/mvcc/cataloger_create_repository.go 59.25% <0.00%> (-3.71%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06806c8...ab24d5b. Read the comment docs.

tzahij
tzahij previously approved these changes Nov 29, 2020
GetEntry(path rocks.Path, tid SSTableID) (*rocks.Entry, error)

// SSTableIterator takes a given SSTable and returns an EntryIterator seeked to >= "from" path
SSTableIterator(tid SSTableID, from rocks.Path) (rocks.EntryIterator, error)
Copy link
Contributor

Choose a reason for hiding this comment

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

I prefere newSSTableIterator

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You got it.

forest/sstable/sstable.go Outdated Show resolved Hide resolved
@tzahij tzahij self-requested a review November 29, 2020 10:27
@tzahij tzahij dismissed their stale review November 29, 2020 10:27

I need the requested changes

@tzahij tzahij merged commit 1cb8a36 into master Nov 29, 2020
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.

5 participants