{"action":"closed","issue":{"url":"https://api.github.com/repos/UniversaBlockchain/universa/issues/3","repository_url":"https://api.github.com/repos/UniversaBlockchain/universa","labels_url":"https://api.github.com/repos/UniversaBlockchain/universa/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/UniversaBlockchain/universa/issues/3/comments","events_url":"https://api.github.com/repos/UniversaBlockchain/universa/issues/3/events","html_url":"https://github.com/UniversaBlockchain/universa/issues/3","id":253093965,"number":3,"title":"глобально блокирующий http сервер вызов \"/network\"","user":{"login":"magum","id":1042529,"avatar_url":"https://avatars3.githubusercontent.com/u/1042529?v=4","gravatar_id":"","url":"https://api.github.com/users/magum","html_url":"https://github.com/magum","followers_url":"https://api.github.com/users/magum/followers","following_url":"https://api.github.com/users/magum/following{/other_user}","gists_url":"https://api.github.com/users/magum/gists{/gist_id}","starred_url":"https://api.github.com/users/magum/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/magum/subscriptions","organizations_url":"https://api.github.com/users/magum/orgs","repos_url":"https://api.github.com/users/magum/repos","events_url":"https://api.github.com/users/magum/events{/privacy}","received_events_url":"https://api.github.com/users/magum/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-08-26T13:49:08Z","updated_at":"2017-08-27T08:44:51Z","closed_at":"2017-08-27T08:44:51Z","body":"https://github.com/UniversaBlockchain/universa/blob/d5d9f74f1487cc3fffa05bb6b20fad71efe19885/universa_core/src/main/java/com/icodici/universa/node/network/ClientEndpoint.java#L202\r\n\r\nтут изза synchronized получается что \"/network\" это глобально блокирующий весь http сервер  вызов. и непонятно ради чего.\r\nBinder networkDirectory это закэшированный ответ. кэшируется он один раз при первом запросе \"/network\" и больше не предполагается меняться. Я так понял это потому что ростер в рантайме не изменяется. Тк по факту ростер только читается при старте из конфига.\r\nте получается ростер и networkDirectory после создания только реадонли.\r\nтогда зачем блокировать весь хттп сервер? Можно все на старте сделать - создали ростер, \r\nзакэшировали сразу ответ в networkDirectory и возвращаем его на каждый запрос \"/network\" без блокировок.\r\n\r\nесли же ростер в будующем будет изменяемый в рантайме, то скоуп synchronized можно уменьшить до конкетного переключения кэша на новое содержимое:\r\nsynchronized (this) {\r\n  networkDirectory = network;\r\n}\r\nа предварительно подготавливать содержимое network можно без блокировок, ведь ростер concurenthashmap\r\n\r\nно возможно вызов \"/network\" редкий и не стоит заморачиваться"}}
{"actor":{"display_login":"sergeych"}}