icingadb icon indicating copy to clipboard operation
icingadb copied to clipboard

Published 20 hours ago • verified publisher icon Icinga

Micro optimizations

Open lippserd opened this issue 3 years ago • 5 comments

lippserd avatar Jul 19 '21 19:07 lippserd

@lippserd What's the future plan for this PR?

julianbrost avatar Oct 22 '21 10:10 julianbrost

@lippserd What's the future plan for this PR?

Honestly, I don't know yet but I asked @Al2Klimov to test whether this PR does anything in terms of performance, cpu and memory usage.

lippserd avatar Oct 22 '21 11:10 lippserd

TODO

  • [ ] throw away everything not to be merged anyway (e.g. exit after config sync)
  • [ ] make CI/CD happy

Al2Klimov avatar Oct 22 '21 11:10 Al2Klimov

Tip: git push -f commit by commit sleeping 10s inbetween, so we see whose commits broke the CI/CD.

Al2Klimov avatar Oct 22 '21 11:10 Al2Klimov

Tip: git push -f commit by commit sleeping 10s inbetween, so we see whose commits broke the CI/CD.

The new types from your commits don’t have the database interfaces implemented and thus the delete fails if I remember correctly.

lippserd avatar Oct 22 '21 12:10 lippserd