NetworkImpl.NetworkImplBuilder should not allow to reassign id()

[leventov]

NetworkImpl.NetworkImplBuilder should not allow to reassign id() because it will have no effect on the actual id.

Also, strictly speaking, close() may not see the id value if called from another thread than getId(). To ensure that, close() method should be synchronized as well as getId(). I'm not sure how relevant this is, but since getId() is synchronized itself it might be.

[bsideup]

Hi @leventov,

Thanks for reporting. The issue is up for grabs in case somebody wants to convert the suggestions into a PR.

[Mariochino]

#2450

[quincy]

I've submitted a PR to resolve this issue. #2807

[Nicklas2751]

This issue can be closed since #2807 is merged. Right?