DialectFunctionDispatcher does not support nesting of decorators
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
oslo.db |
Fix Released
|
Medium
|
Mike Bayer |
Bug Description
the DialectFunction
@_init_
@_init_
def _init_connectio
# ...
Because the dispatch_for() decorator returns the dispatcher, and not the function (necessary so that the dispatcher remains available even decorating a same-named function), nesting above does not work automatically, you instead get a recursion overflow:
File "/Users/
self.
File "/Users/
return self._dispatch_
File "/Users/
if self._invoke_fn(fn, arg, kw) is not None:
File "/Users/
return fn(*arg, **kw)
File "/Users/
self.
File "/Users/
return self._dispatch_
File "/Users/
The decorator should anticipate this condition and adapt for it.
Changed in oslo.db: | |
assignee: | nobody → Mike Bayer (zzzeek) |
Changed in oslo.db: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in oslo.db: | |
status: | Confirmed → Fix Released |
Do we need to fix this in Mitaka zzzeek?