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.

Sign up for GitHub

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

Plat 249 reduce rerun time #1165

Merged
merged 17 commits into from
Nov 3, 2022
Merged

Plat 249 reduce rerun time #1165

Show file tree
Hide file tree
Changes from 16 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
59b12bc
Return the total number of transactions
bradsawadye Aug 24, 2022
f2d1dab
Reduce the time taken for a transaction to be rerun
bradsawadye Aug 24, 2022
71245e3
Add endpoint for creating rerun tasks for bulk reruns
bradsawadye Sep 9, 2022
57d0fb9
Process 3 tasks concurently and check whether task is unpaused before…
bradsawadye Sep 9, 2022
cdad2da
Add tests for the rerun endpoint and fix breaking tests
bradsawadye Sep 9, 2022
febb063
Add the default task transactions array length
bradsawadye Sep 9, 2022
06cd5ba
Bump the project service
bradsawadye Sep 9, 2022
cb129a2
Correct the event projection
bradsawadye Oct 7, 2022
0074de8
Change variable name to be camelcase for consistency
bradsawadye Oct 10, 2022
6ce4d69
Correct the project version
bradsawadye Oct 10, 2022
0ad1600
Only return the total number of transactions for a bulk rerun
bradsawadye Oct 10, 2022
c4f977f
Refactor
bradsawadye Oct 10, 2022
d314e72
Add more tests
bradsawadye Oct 10, 2022
c0dd351
Make the variables used in recursive function global
bradsawadye Oct 10, 2022
15671f4
Ensure function for creating rerun is only called when there are tran…
bradsawadye Oct 10, 2022
89ca88a
Add more tests
bradsawadye Oct 10, 2022
0236a38
Update test/unit/tasksTest.js
bradsawadye Oct 31, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions config/default.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,15 +39,17 @@
"maxPayloadSizeMB": 50,
"truncateSize": 15000,
"truncateAppend": "\n[truncated ...]",
"authenticationTypes": ["token"]
"authenticationTypes": ["basic", "token"]
bradsawadye marked this conversation as resolved.
Show resolved Hide resolved
},
"rerun": {
"httpPort": 7786,
"host": "localhost",
"processor": {
"enabled": true,
"pollPeriodMillis": 2000
}
},
"taskTransactionsLength": 50,
"activeConcurrentTasks": 3
},
"tcpAdapter": {
"httpReceiver": {
Expand Down
2 changes: 1 addition & 1 deletion package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "openhim-core",
"description": "The OpenHIM core application that provides logging and routing of http requests",
"version": "7.0.2",
"version": "7.1.0",
"main": "./lib/server.js",
"bin": {
"openhim-core": "./bin/openhim-core.js"
Expand Down
2 changes: 1 addition & 1 deletion src/alerts.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ const findTransactionsMaxRetried = (channel, alert, dateFrom, callback) =>
status: 500,
autoRetryAttempt: channel.autoRetryMaxAttempts
},
{transactionID: 'transactionID'}
{transactionID: 1}
bradsawadye marked this conversation as resolved.
Show resolved Hide resolved
)
// .hint({created: 1})
.exec((err, transactions) => {
Expand Down
Loading