Nodejs pre release 5.9.0 #591
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Nodejs-Release-5.8.0
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-2572
Added exp.recordSize
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
Added maps.create method
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
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## stage #591 +/- ##
==========================================
+ Coverage 91.25% 91.55% +0.30%
==========================================
Files 69 69
Lines 2229 2250 +21
==========================================
+ Hits 2034 2060 +26
+ Misses 195 190 -5
☔ View full report in Codecov by Sentry. |
Updated node-gyp version in windows github action
changed npm install to npm ci in windows github action
3.12 breaks node-gyp on windows.
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
realmgic
previously approved these changes
Nov 6, 2023
lib/client.js
Outdated
| * | ||
| * // INSERT HOSTNAME AND PORT NUMBER OF AEROSPIKE SERVER NODE HERE! | ||
| * // INSERT HOSTNAME AND PORT NUMBER OF AEROPSIKE SERVER NODE HERE! |
artanderson
previously approved these changes
Nov 7, 2023
Ran npm audit fix. Removed Dockerfiles Added README.md to docker folder explaining where files went.
DomPeliniAerospike
dismissed stale reviews from artanderson and realmgic
via
60a7ddd
mcoberly2
approved these changes
Nov 7, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.