Closed michaeldinzinger closed 1 year ago
A simpler approach would be to force FrontierServices to have a constructor containing (String host, String port) so that these values would be set before the Hearbeat thread starts working. Alternatively, we could introduce a new method start() for FrontierServices and call that after service.setHostAndPort(host, port); in the server class. We'd separate the instantiation from the actual starting of the various subthreads. What do you think @michaeldinzinger?
I like the first idea to add host and port to the constructor. It should fix the problem and is easy to implement and understand
(Don't be irritated that I uploaded two commits. First, I looked at the idea to add the start()
method, but reverted the changes and implemented the idea to modify the constructor; it seems to be better coding style)
However, it has to be considered that the constructor has to be changed in all implementations, also in the opensearch implementation as soon as it would use this newer version of the urlfrontier code, in the future
Perfect, thanks @michaeldinzinger
Signed-off-by: Michael Dinzinger michael.dinzinger@uni-passau.de
Thanks for contributing to URL Frontier, your efforts are appreciated!
Developer Certificate of Origin
By contributing to URL Frontier, you accept and agree to the following terms and conditions (the Developer Certificate of Origin) for your present and future contributions submitted to URL Frontier. Please refer to the Developer Certificate of Origin section in
CONTRIBUTING.md
for details.Before opening a PR, please check that:
Thanks!