Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consolidate route code #341

Closed
brycelelbach opened this issue Jul 10, 2012 · 7 comments
Closed

Consolidate route code #341

brycelelbach opened this issue Jul 10, 2012 · 7 comments

Comments

@brycelelbach
Copy link
Member

[reported by hkaiser] [Trac time Tue Feb 28 16:15:45 2012] We currently have duplicated code which is a maintenance problem.

The duplication needs to be consolidated in (at least):

  • apply
  • futures
@brycelelbach
Copy link
Member Author

[comment by blelbach] [Trac time Fri Apr 6 02:34:44 2012] Vinay, status on this?

@brycelelbach
Copy link
Member Author

[comment by vamatya] [Trac time Fri Apr 6 02:48:13 2012] It has not been looked into. I'll start on it.

@brycelelbach
Copy link
Member Author

[comment by hkaiser] [Trac time Tue Apr 24 11:33:05 2012] This should be done by moving the routing code to the parcel layer. This aditionally implies we come up with some metric allowing to decide whether to route or not.

Another solution to replace the current scheme with routing alltogether would be to make sure that routing does not introduce any significant serialization overheads for large (payload-) parcels. One possibility would be to store parcels as payload in the wrapping routing parcel in form of a unstructured byte sequence. This would prevent deserialization/serialization of the embedded parcel on the AGAS server node.

@brycelelbach
Copy link
Member Author

Vinay, can you please look into this? This has been opened for more than 6 months.

@vamatya
Copy link
Member

vamatya commented Aug 22, 2012

Bryce, I'm working on metrics. As soon as I'm done with metric system, I'll consolidate the route code

@brycelelbach
Copy link
Member Author

Please make this a priority, it's important that you wrap up this work.

@ghost ghost assigned vamatya Aug 22, 2012
@hkaiser
Copy link
Member

hkaiser commented Apr 10, 2013

This has been fixed while implementing the distributed AGAS (see #177: Implement distributed AGAS)

@hkaiser hkaiser closed this as completed Apr 10, 2013
@ghost ghost assigned hkaiser Apr 11, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants