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

OAK-11525 Update fullGC configuration for audit logging into Mongo collection #1

Merged
merged 8 commits into from
Mar 4, 2025

Conversation

daniancu
Copy link
Owner

@daniancu daniancu commented Feb 26, 2025

Add OSGi configuration to enable/disable full GC audit bin (default disabled)
Add -fullGCAuditLoggingEnabled option for oak-run (default disabled)

public static final String GC_COLLECTED_AT = "_gcCollectedAt";
private static final Logger LOG = getLogger(MongoFullGcNodeBin.class);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use LoggerFactory.getLogger(MongoFullGcNodeBin.class) instead of getLogger(MongoFullGcNodeBin.class)

@@ -112,8 +112,13 @@ public class MongoVersionGCSupport extends VersionGCSupport {
*/
private final int batchSize = Integer.getInteger(
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use ... SystemPropertySupplier

@Joscorbe
Copy link

Joscorbe commented Mar 4, 2025

The "into" branch is on your private repository, which I can't access, so can't merge it. You can probably integrate this PR into apache#2038 and everything will be merged together into Oak.

@daniancu daniancu merged commit 756e189 into OAK-11444 Mar 4, 2025
1 check passed
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.

None yet

3 participants