[
https://issues.apache.org/jira/browse/ARIA-414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16276599#comment-16276599
]
ASF GitHub Bot commented on ARIA-414:
-------------------------------------
aviyoop commented on a change in pull request #209: ARIA-414 Current events
handler mechanism relies on sqlalchemy
URL:
https://github.com/apache/incubator-ariatosca/pull/209#discussion_r154609888
##########
File path: tests/orchestrator/workflows/executor/test_executor.py
##########
@@ -28,45 +28,48 @@
import aria
from aria.modeling import models
from aria.orchestrator import events
+from aria.orchestrator.workflows.core import events_handler
from aria.orchestrator.workflows.executor import (
thread,
process,
# celery
)
import tests
+from .. import helpers
from . import MockContext
+
Review comment:
remove (if this is not the only change)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Current events handler mechanism relies on sqlalchemy
> -----------------------------------------------------
>
> Key: ARIA-414
> URL: https://issues.apache.org/jira/browse/ARIA-414
> Project: AriaTosca
> Issue Type: Story
> Reporter: Maxim Orlov
> Assignee: Maxim Orlov
> Priority: Minor
>
> The Current events handler mechanism relies to heavily on sqlalchemy MAPI.
> The mechanism just updates the state which is save in the sqllachemy session.
> The mechanism should be more generic and support different MAPI
> implementations.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
