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

Nodejs-Release-5.9.0 #594

Merged
merged 34 commits into from
Nov 7, 2023
Merged

Nodejs-Release-5.9.0 #594

merged 34 commits into from
Nov 7, 2023

Conversation

DomPeliniAerospike
Copy link
Collaborator

No description provided.

CLIENT-2629
Added logic to ignore AEROSPIKE_BATCH_FAILED in cases where records are returned for all batch commands
Fixed AEROSPIKE_BATCH_FAILED error issue
CLIENT-2592
Added exp.expWriteFlags and exp.expReadFlags
Added exp.expWriteFlags and exp.expReadFlags
CLIENT-2587
Added Blob Secondary index support
Added client.createBlobIndex
Fixed Javascript Maps to place non-string indexes on server
Added Bolb Secondary Index support.
CLIENT-2617
Added maps.create method
Added persitIndex as a parameter
use "npm run lint"
Added additional logic to batchRemove test to clean records in which batchRemove failed
CLIENT-1421
Added maxErrorRate and errorRateWindow configurations.
Fixed bug with tenderInteval
Added maxErrorRate and errorRateWindow configurations
Added/modified Client examples for batch, maps.create,  expression operations.
Added context as a parameter to maps.create.
Modified testing config to set max_error_rate = 0
Modifed testing to omit 7.0 features when server is a lower version.
Added documentation.
Removed unnecessary memory allocation and print statements in query.cc
Disabled blob sindex testing on server versions less than 7.0
Updated node-gyp version in windows github action
changed npm install to npm ci in windows github action
Fixed batchRemove parameter to keys rather than records
Changed the aerospike-client-c.ini file to current package version.
Changed reference to records to new parameter name keys
Ran npm audit fix.
Removed Dockerfiles
Added README.md to docker folder explaining where files went.
Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (a969a6b) 89.86% compared to head (3770cc4) 91.55%.
Report is 23 commits behind head on master.

❗ Current head 3770cc4 differs from pull request most recent head 7f89eec. Consider uploading reports for the commit 7f89eec to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #594      +/-   ##
==========================================
+ Coverage   89.86%   91.55%   +1.68%     
==========================================
  Files          68       69       +1     
  Lines        2221     2250      +29     
==========================================
+ Hits         1996     2060      +64     
+ Misses        225      190      -35     
Files Coverage Δ
lib/aerospike.js 97.33% <100.00%> (+0.03%) ⬆️
lib/config.js 92.45% <100.00%> (+0.61%) ⬆️
lib/error.js 97.14% <ø> (ø)
lib/exp.js 100.00% <100.00%> (ø)
lib/exp_lists.js 89.09% <100.00%> (+21.81%) ⬆️
lib/exp_maps.js 82.53% <100.00%> (+30.15%) ⬆️
lib/exp_operations.js 92.30% <ø> (ø)
lib/maps.js 100.00% <100.00%> (ø)
lib/bin.js 80.00% <80.00%> (ø)
lib/filter.js 98.03% <50.00%> (-1.97%) ⬇️
... and 1 more

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DomPeliniAerospike DomPeliniAerospike merged commit c44ebdb into master Nov 7, 2023
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.

2 participants