reTHINK-project / dev-runtime-core

Javascript Runtime
Apache License 2.0
3 stars 3 forks source link

Runtime Registry: P2P Protostubs support #134

Closed pchainho closed 7 years ago

pchainho commented 7 years ago
vitormsilva commented 7 years ago

Hi, I have a problem with the P2P resolve. When a hyperty is registred, and the messages will be send to other hyperty, i always have checkHypertyP2PHandler don't find any hyperty.

screenshot from 2016-12-06 17-15-59

Could you check the registerHyperty and from step 6 to step 22 here.

If you need help, we can try to help you..

vitormsilva commented 7 years ago

I found other problem.. I don't know if it's related to, but seems have some "fake" data, in these case to register dataObjects.

screenshot from 2016-12-07 12-19-24

tiagolb commented 7 years ago

That error message comes from a reject in the checkHypertyP2pHandler function which is responsible for searching in the local cache for a p2phandler and if no p2phandler is foound it should trigger a discovery process (step 9 to step 14 of the spec).

But this discovery process is not yet done. I'm looking into it.

tiagolb commented 7 years ago

@BernardoMG The function to be modified in order to support the discovery service for finding the p2pHandler is called checkHypertyP2pHandler in line 800 of the Registry.js. There's a TODO comment in that line where the discovery process must be triggered.

BernardoMG commented 7 years ago

@tiagolb Currently I'm working on some issues on Domain Registry, i will see that as soon as I can.

BernardoMG commented 7 years ago

@vitormsilva @tiagolb Already implemented the discovery process in checkHypertyP2PHandler. The changes are in dev-p2pbranch. It's necessary to use the branch connector-release-0.6.0 in dev-msg-node-vertx.

pchainho commented 7 years ago

pls in case they are complete, update the the tasks list above and close the issue if everything is done. Otherwise comment here what is missing

pchainho commented 7 years ago

Done