-
Notifications
You must be signed in to change notification settings - Fork 47
ARIA-42-Generic-ctx-serialization-mechanism #68
Conversation
77165a4
to
4c5126e
Compare
aria/storage/core.py
Outdated
def __init__(self, api_cls, api_kwargs=None, items=(), **kwargs): | ||
def __init__(self, | ||
api_cls, | ||
items=(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add api_kwargs and move init, init_kwargs to ModelStorage (and rename init to session_initiator)
aria/storage/sql_mapi.py
Outdated
|
||
from sqlalchemy import create_engine |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix imports
aria/storage/sql_mapi.py
Outdated
path=os.path.join(base_dir, filename)) | ||
engine_kwargs = {} | ||
else: | ||
uri = 'sqlite:///:memory:' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove in memory option to tests only
tests/mock/context.py
Outdated
|
||
deployment_id = create_simple_topology_two_nodes(model_storage) | ||
model_storage = aria.application_model_storage( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
be consistent with initiator passing
|
||
def __init__(self, *args, **kwargs): | ||
def __init__(self, *args, **kwargs): # pylint: disable=super-init-not-called |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add serialize function
|
||
def __init__(self, *args, **kwargs): | ||
def __init__(self, *args, **kwargs): # pylint: disable=super-init-not-called |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
Can one of the admins verify this patch? |
aria/storage/core.py
Outdated
@@ -66,10 +67,10 @@ def __init__(self, | |||
self.registered = {} | |||
self._initiator = initiator | |||
self._initiator_kwargs = initiator_kwargs or {} | |||
self._api_kwargs = api_kwargs or {} | |||
self._additional_api_kwargs = {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add information about initiator interface
aria/storage/core.py
Outdated
@@ -66,10 +67,10 @@ def __init__(self, | |||
self.registered = {} | |||
self._initiator = initiator | |||
self._initiator_kwargs = initiator_kwargs or {} | |||
self._api_kwargs = api_kwargs or {} | |||
self._additional_api_kwargs = {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unpicklable
@@ -65,3 +65,14 @@ def release_sqlite_storage(storage): | |||
session.close() | |||
for engine in set(mapi._engine for mapi in mapis): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dont forget to revisit mock context
3436987
to
6311e45
Compare
6311e45
to
1498ad3
Compare
No description provided.