Community detection memory tracking

[imilinovic]

Description

Community detection memory tracking was not working properly when multiple threads were used so it was fixed.

Add thread memory tracking to community detection.

Requires PR436 and PR1631 before merging

Note to reviewer (!!!): Review with suffix ?w=1 to ignore whitespaces changes which are automatically made by IDE (and there are a lot of them). Link for reviewing

Pull request type

• [X] Bugfix
• [ ] Algorithm/Module
• [ ] Feature
• [ ] Code style update (formatting, renaming)
• [ ] Refactoring (no functional changes, no api changes)
• [ ] Build related changes
• [ ] Documentation content changes
• [ ] Other (please describe):

Release note:

Community detection now respects memory limits and doesn't crash database.

Related issues

closes #379 closes #219 closes https://github.com/memgraph/memgraph/issues/955

######################################

Reviewer checklist (the reviewer checks this part)

Module/Algorithm

• [ ] Core algorithm/module implementation
• [ ] Query module implementation
• [ ] Unit tests
• [ ] End-to-end tests
• [ ] Code documentation
• [ ] README short description
• [ ] Documentation on memgraph/docs
• [ ] Update GQLALchemy signatures in query builder using query module signature generator

######################################

[gitbuda]

@imilinovic, quick question, what's the issue related? Can you link it? 👀

[sonarqubecloud[bot]]

Quality Gate failed

Failed conditions
21.8% Duplication on New Code (required ≤ 3%)
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

[imilinovic]

I think the test to check that this works can't be done because of the following:

Memory limit is always being respected but the problem was that memory counting was wrong. The only way would be to compare it with the system memory (or see if the system crashes when setting the limit close to available RAM on the machine) which I don't think is possible at least not with the current MAGE testing infrastructure. @antepusic @DavIvek

I think sonarcloud can just be ignored here and merge if all other tests pass.