Skip to content

Add UNF flag for SORTABLE fields#2188

Merged
ashtul merged 3 commits intomasterfrom
MOD1357
Aug 23, 2021
Merged

Add UNF flag for SORTABLE fields#2188
ashtul merged 3 commits intomasterfrom
MOD1357

Conversation

@ashtul
Copy link
Copy Markdown
Contributor

@ashtul ashtul commented Aug 18, 2021

Replated to MOD-1357

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 18, 2021

Codecov Report

Merging #2188 (b050f32) into master (8e92a7b) will increase coverage by 0.01%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2188      +/-   ##
==========================================
+ Coverage   80.43%   80.45%   +0.01%     
==========================================
  Files         150      150              
  Lines       22125    22128       +3     
==========================================
+ Hits        17796    17802       +6     
+ Misses       4329     4326       -3     
Impacted Files Coverage Δ
src/document.c 73.12% <57.14%> (-0.17%) ⬇️
src/sortable.c 63.20% <100.00%> (ø)
src/spec.c 84.73% <100.00%> (+0.02%) ⬆️
src/fork_gc.c 57.37% <0.00%> (+0.54%) ⬆️

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 8e92a7b...b050f32. Read the comment docs.

@ashtul ashtul merged commit aa9808d into master Aug 23, 2021
@ashtul ashtul deleted the MOD1357 branch August 23, 2021 08:58
ashtul added a commit that referenced this pull request Aug 23, 2021
* Add UNF flag for SORTABLE fields

* add test

(cherry picked from commit aa9808d)
ashtul pushed a commit that referenced this pull request Aug 24, 2021
…xity to O(1) (#2153) (#2201)

* Add UNF flag for SORTABLE fields (#2188)

* Add UNF flag for SORTABLE fields

* add test

(cherry picked from commit aa9808d)

* Document SORTABLE UNF

* Improve FT.INFO complexity to O(1) (#2153)

* Improve FT.INFO complexity to O(1)

* split cardinality and size for MemUsage

* Per meir's review

* per review

(cherry picked from commit e1a8af5)

* remove json test

Co-authored-by: Emmanuel Keller <emmanuel.keller@redislabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants