-
- Notifications
You must be signed in to change notification settings - Fork 189
Description
The whole benchmark is broken by 768bfa5 since the SockAddr is not used by the benchmark and no "legacy" signature (with address as string) is still available in the python API for bootstraping 😒.
This is the case for benchmark, but also all python code which have already been written before 768bfa5. For instance, the pingpong script is broken. I think that some of ring's dht small swarm is using python bindings also. This is dangerous.
This kind of changes should be made more carefuly in the future since we never know who may use the python bindings also and we don't want to just drop support for API as fundamental as DhtRunner.bootstrap. I suggest, a legacy signature should be provided. If we want to deprecate it, then we could add a message on standard error specifying the matter, but anyhow I don't think it's good idea to remove this signature before a long time (couple of weeks or months).