Task_2_1_2#15
Conversation
|
|
galinabykova
left a comment
galinabykova
left a comment
There was a problem hiding this comment.
в твоей версии slave знают host мастера, но задание предполагает, что они будут находить друг друга в одной подсети. Это нужно исправить
| * @param taskId . | ||
| * @param hasNonPrime . | ||
| */ | ||
| public record TaskResult(int taskId, boolean hasNonPrime) implements Serializable { |
There was a problem hiding this comment.
какие есть плюсы и минусы у выбранного тобой формата передачи данных (бинарная сериализация)
There was a problem hiding this comment.
ну если вопрос про сравнение бинарной сериализации и какого-нибудь JSONа, то с бинарной проще работать, потому что не надо никаких парсеров, ну и потенциально это компактнее, хотя наверное не всегда
А из минусов, из-за такой сериализации я вынужден пользоваться джавой на обоих концах, потому что объект джавовский. Ну и где-то пишут про то, что потенциально злоумышленник может зашить туда что-то такое, что сможет исполнить то, что не надо на моей машине
| for (var task : taskQueue) { | ||
| System.out.print(task.taskId() + " "); | ||
| } |
There was a problem hiding this comment.
мне кажется, такой лог занимает очень много времени
There was a problem hiding this comment.
это да, просто с ним нагляднее, как оно работает, но могу убрать
| } | ||
| System.out.println(); | ||
| tasksLeft--; | ||
| lock.notifyAll(); |
There was a problem hiding this comment.
тебе нужно будить даже если составного числа не было?
There was a problem hiding this comment.
действительно, можно будить только на true или если таски кончились
| private final AtomicBoolean globalResult = new AtomicBoolean(false); | ||
| private final AtomicBoolean isDone = new AtomicBoolean(false); | ||
| private final List<Thread> workerHandlers = new CopyOnWriteArrayList<>(); | ||
| private final Object lock = new Object(); |
There was a problem hiding this comment.
почему ты решил использовать монитор нового объекта, а не монитор самого Master?
There was a problem hiding this comment.
я не подумал мониторить мастер)
There was a problem hiding this comment.
нужен тест, показывающий, что после отсоединения одного из slave, его работа будет досчитана
|
|
2 participants
No description provided.