search

elixir-cloud-aai / proTES

Proxy service for injecting middleware into GA4GH TES requests
Apache License 2.0
5 stars 6 forks source link

feat: distance-based task distribution logic #119

Closed Ayush5120 closed 1 year ago

Ayush5120 commented 1 year ago

Added task distribution logic, that computes the total distance between the set of input URIs (from the incoming TES task) and the available TES instances, then relays the task to the closest TES instance.

uniqueg commented 1 year ago

Hi @Ayush5120. Thanks a lot for the great work :)

I am merging now, as I guess you have addressed at least most of the comments (and I have opened #124). If there are any more issues popping up during testing, we can open individual issues for them and fix them :)

In the future, I think it's best if you resolve open conversations yourself if you feel confident that you have addressed them non-controversially. Otherwise, please leave a comment, either describing how you addressed it, or why you did not address it. That makes reviewing a whole lot easier.