radical-cybertools / radical.saga

A Light-Weight Access Layer for Distributed Computing Infrastructure and Reference Implementation of the SAGA Python Language Bindings.
http://radical-cybertools.github.io/saga-python/
Other
83 stars 34 forks source link

Remove redis-adaptor loading #803

Closed mtitov closed 4 years ago

mtitov commented 4 years ago

Should some other adaptors be removed (comment out in config) from the loading as well?

p.s. Redis adaptor is not removed completely, just not loading (if there will be a request for it, then it can be turned on and some it parts should be updated according to the latest version of RU)

mtitov commented 4 years ago

@andre-merzky I found this inconsistency (since it was almost at the top, but didn't check everything there, may be it is the only one):

import radical.utils.threads as rut
...
rut.Thread.__init__(self, self.work)

and radical.utils.threads doesn't have Thread anymore

edit: module radical.saga.adaptors.redis.redis_namespace