[ 
https://issues.apache.org/jira/browse/ARIA-414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16276600#comment-16276600
 ] 

ASF GitHub Bot commented on ARIA-414:
-------------------------------------

Github user aviyoop commented on a diff in the pull request:

    https://github.com/apache/incubator-ariatosca/pull/209#discussion_r154609888
  
    --- Diff: 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
     
     
    +
    --- End diff --
    
    remove (if this is not the only change)


> 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)

Reply via email to