On Fri, 2010-12-03 at 12:53 +0100, lmartinc(a)redhat.com wrote:
git: [patch aggregator] first impl. of adapter for condor backend.
Hey guys,
probably this is going to be very controversial patch. This patch should allow people
to be able to use different job-backend if they chose to. I have refactored Condor
backend
to be easily subtituteable via Adapter pattern. The implementation is just begining, but
if at least some of you will like it I will continue to provide at least one other
job-backend (DelayedJobs).
The intention isn't to replace Condor, but make it optional and not hardcoded.
If there will be no reply or dicussion I will drop this patch.
Thanks for your feedback.
You're right, it is controversial :)
I really hope we do not have alternative backends any time soon.
There's only a handful of people who test/use the aggregator end to end.
If some of these people decide it would be easier to not use condor,
it's just that many fewer people testing critical production code
paths.
I really cannot stress that enough. It's all about testing.. it needs
testing, lots of it, and this just enables it to be tested less.
If there is something difficult about the installation we need to
address that (and we are addressing some aspects of it).
To play devils advocate, I could easily create something to replace the
web UI as well, but I don't, because that's a huge part of the product
and it too needs testing and I hope everyone uses/tests it. You see
what I'm saying?
Maybe once the whole thing is more mature this would be a good idea, but
for now I don't think it's appropriate.
Ian