Skip to content
This repository has been archived by the owner on Sep 27, 2022. It is now read-only.

New parallel branch Кривчанский 395 #549

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

krivchnik
Copy link
Contributor

Кривчанский 395

Can't wait to fix all checkstyle errors
Коммит не той версии это эпично.
Вообще стоит отметить что он и создавал файл при отсутствии и корректно
работал с пустым файлом, но писал "предупреждение" в поток ошибок, да.
some small commit
@krivchnik krivchnik changed the title New parallel branch New parallel branch Кривчанский 395 Dec 15, 2014
@krivchnik
Copy link
Contributor Author

старый Parallel #327

@Before
public void beforeTest() {
factory = new DatabaseFactory();
provider = factory.create("C:\\temp\\database_test");
Copy link
Collaborator

Choose a reason for hiding this comment

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

Так нельзя, используй временную папку

Если я правильно понял -- более гранулярный означал менее общий. Добавил
удаление папки после её создания.  Добавил локи там где их не хватало.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants