Skip to content

Feature/bjs2 96747#3468

Open
wersatil wants to merge 5 commits intophoenix-master-stream12from
feature/bjs2-96747
Open

Feature/bjs2 96747#3468
wersatil wants to merge 5 commits intophoenix-master-stream12from
feature/bjs2-96747

Conversation

@wersatil
Copy link
Copy Markdown

No description provided.

@github-actions
Copy link
Copy Markdown

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

@github-actions
Copy link
Copy Markdown

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

…де конкретно ошибка. Повторный ПР в качестве попытки обнаружить проблему.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 3, 2025

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

…де конкретно ошибка. Повторный ПР в качестве попытки обнаружить проблему.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 3, 2025

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

@RequiredArgsConstructor
@ConditionalOnProperty(value = "services.s3.isMocked", havingValue = "false")
public class S3ServiceImpl implements S3Service {
private final AmazonS3 s3Client; //Именно в нем проблема, он не создается!
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

"Именно в нем проблема, он не создается!" - сейчас ошибка ушла или осталась проблема ?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Это я пометил для себя полчаса назад, чтобы на сегодняшнем созвоне обсудить. ) Аяз говорил, что у него тоже какая-то проблема с Amazon, я сделал вывод, что это - какая-то глобальная ошибка, которая нам с ним не под силу, и нужно задать вопрос на сегодняшнем созвоне. Чтобы тебе наглядно показать, что к чему, я сделал себе пометку ) Проблема в создании этого бина. Если эту строчку убрать, приложение запускается, но, конечно же, Амазон не срабатывает )

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Кстати, сборка в ПР тоже заваливается "без объяснений". Я предполагаю, что тут та же самая проблема.

private String bucketName;

@Override
@Transactional
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Важный вопрос: расскажи почему здесь используешь транзакцию и уверен ли что она сработает?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Пару недель назад вы с Лилей говорили, что нужно ставить @transactional на все методы, которые делают хотя бы один поход в базу с попыткой изменить хотя бы одно поле. Поэтому я ее поставил тут. Что в этой задаче хоть что-то сработает, к сожалению, не уверен )) Пока не могу протестировать, из-за проблемы с бином AmazonS3.

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