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

IGNITE-24309 Decrease number of slots in HeapLockManager in tests #5118

Closed
wants to merge 3 commits into from

Conversation

rpuch
Copy link
Contributor

@rpuch rpuch commented Jan 24, 2025

*
* @param nodeName Name of the node. Must not be {@code null}.
* @param configStr Optional node configuration.
* @return Completable future that resolves into an Ignite node after all components are started and the cluster initialization is
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing param workDir here.

@@ -206,4 +243,8 @@ private static ConfigDocument applyTestDefault(ConfigDocument document, String p
return document.withValueText(path, value);
}
}

private static String toDoubleQuotedString(Object obj) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's add this to StringUtils, 'cause we have quite a lot of places where we can have a use of this method.

@rpuch
Copy link
Contributor Author

rpuch commented Feb 21, 2025

Closing as this is not relevant anymore. However, part of the work that was done in this PR was moved to #5265.

@rpuch rpuch closed this Feb 21, 2025
@rpuch rpuch deleted the ignite-24309 branch February 21, 2025 10:53
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.

2 participants