Skip to content

Commit

Permalink
abort live streams using true instead of error
Browse files Browse the repository at this point in the history
  • Loading branch information
staltz committed Jun 28, 2022
1 parent ff6d8eb commit 869133a
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
5 changes: 0 additions & 5 deletions errors.js
Expand Up @@ -42,10 +42,6 @@ function appendLargerThanBlockErr() {
return new Error('Data to be appended is larger than block size')
}

function streamClosedErr() {
return new Error('async-append-only-log stream is closed')
}

function appendTransactionWantsArrayErr() {
return new Error('appendTransaction expects first argument to be an array')
}
Expand All @@ -66,7 +62,6 @@ module.exports = {
deletedRecordErr,
delDuringCompactErr,
appendLargerThanBlockErr,
streamClosedErr,
appendTransactionWantsArrayErr,
unexpectedTruncationErr,
}
3 changes: 1 addition & 2 deletions index.js
Expand Up @@ -18,7 +18,6 @@ const {
outOfBoundsOffsetErr,
delDuringCompactErr,
appendLargerThanBlockErr,
streamClosedErr,
appendTransactionWantsArrayErr,
unexpectedTruncationErr,
} = require('./errors')
Expand Down Expand Up @@ -514,7 +513,7 @@ module.exports = function AsyncAppendOnlyLog(filename, opts) {
function close(cb) {
onDrain(function closeAfterHavingDrained() {
onDeletesFlushed(function closeAfterDeletesFlushed() {
for (const stream of self.streams) stream.abort(streamClosedErr())
for (const stream of self.streams) stream.abort(true)
self.streams = []
raf.close(cb)
})
Expand Down

0 comments on commit 869133a

Please sign in to comment.