search

redis / redis-om-python

Object mapping, and more, for Redis and Python
MIT License
1.12k stars 112 forks source link

Updating default page size to 1000 from 10 #282

Closed simonprickett closed 2 years ago

simonprickett commented 2 years ago

The default page size for all is supposed to be 1000 not 10, for performance reasons. When I changed the constant DEFAULT_PAGE_SIZE a while back, I seem to have forgotten to have changed it here. This fixes that.

codecov-commenter commented 2 years ago

Codecov Report

Merging #282 (04438c1) into main (d604797) will increase coverage by 0.06%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #282      +/-   ##
==========================================
+ Coverage   77.27%   77.33%   +0.06%     
==========================================
  Files          12       12              
  Lines        1166     1165       -1     
==========================================
  Hits          901      901              
+ Misses        265      264       -1
Flag Coverage Δ
unit 77.33% <100.00%> (+0.06%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
aredis_om/model/model.py 86.38% <100.00%> (+0.09%) :arrow_up:

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 d604797...04438c1. Read the comment docs.