Skip to content

Conversation

@danikirillov
Copy link
Contributor

From fixed values to % from container memory.

Learn more here - https://m.youtube.com/watch?v=HmlW1W6mxS8

(if i remember correctly, '-XX:+UseContainerSupport' is enabled by default, but just to make it more clear i decided to keep it)

@fey fey merged commit 593a523 into hexlet-components:main Sep 9, 2025
@fey
Copy link
Contributor

fey commented Sep 9, 2025

@danikirillov для джавалина тоже эти опции пригодятся?

@danikirillov
Copy link
Contributor Author

@fey да, в целом для любого жава приложения в контейнере работающего, исключение наверно только native image приложения.

@fey
Copy link
Contributor

fey commented Sep 9, 2025

@danikirillov https://github.com/hexlet-components/java-javalin-blog добавь тогда сюда.

Бтв тут кажется сборка не прошла. Посмотришь, поправишь, пришлешь новый ПР? :)

@danikirillov
Copy link
Contributor Author

danikirillov commented Sep 9, 2025

@fey а, хорошо, добавлю туда, я не увидел жавалин-блога, тк если зайти в 4 проект, ссылка на пример будет вести сюда https://github.com/hexlet-components/java-javalin-example, там и сделал пр вчера.

по поводу красного пайплайна - он такой тк в прошлом коммите был добавлен спотлес, в качестве решения, которое могу сейчас сделать - закоментить спотлесс, тк не уважаю линтеры, ломающие пайплайн. Ну или можно сделать спотлесс:аплай, не уверен, что у меня будет на это время сегодня

@danikirillov
Copy link
Contributor Author

@fey проверил javalin-blog и в нем какой-то старый докерфайл, с 20 жавой, я конечно могу туда добавить параметры, но может лучше обвновить докерфайл целиком? стащить его из жавалин-екзапмпл

@fey
Copy link
Contributor

fey commented Sep 9, 2025

@danikirillov да, будет здорово, если обновишь. Может быть есть на примете студенты, котореы могут помочь тебе. По поводу линтера - закомментить не стоит, лучше поправить.

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