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

Feature/#1140 host logging enhancement optimizations #318

Closed

Conversation

mdebarros
Copy link
Member

@mdebarros mdebarros commented Jan 8, 2020

Changes for mojaloop/project#1140:

  • Updated event-sdk
  • Added partition algorithm config to use murmur2_random which is aligned to official Kafka client implementation

@mdebarros mdebarros self-assigned this Jan 8, 2020
elnyry-sam-k
elnyry-sam-k previously approved these changes Jan 8, 2020
mdebarros and others added 19 commits January 8, 2020 11:58
* added enhanced logging for performance metrics and increased thread pool for node

* updated version
* added enhanced logging for performance metrics and increased thread pool for node

* updated version

* fix log for fulfil

* version change
* added enhanced logging for performance metrics and increased thread pool for node

* updated version

* fix log for fulfil

* version change

* added log statements to remove network lag issues
* added metrics for finer granulation of the notification handler

* fixed await

* moved metrics out of the notification handler and into the called methods
…thub.com:mojaloop/ml-api-adapter into feature/#1140HostLoggingEnhancementOptimizations
* added enhanced logging for performance metrics and increased thread pool for node

* updated version

* fix log for fulfil

* version change

* added log statements to remove network lag issues

* changed logs to error to reduce logging times
version change

* updated package lock
@CLAassistant
Copy link

CLAassistant commented Oct 25, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 5 committers have signed the CLA.

✅ vgenev
✅ oderayi
✅ mdebarros
❌ Valentin
❌ rmothilal


Valentin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
6.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@elnyry-sam-k
Copy link
Member

Closing for now as its been quite some time.

Please reopen in case there's a need to continue this, thank you!

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.

6 participants