[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-07 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r91277691 --- Diff: tests/storage/__init__.py --- @@ -33,10 +35,30 @@ def teardown_method(self): rmtree(self.path, ignore_errors=True)

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-07 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r91277752 --- Diff: tests/storage/__init__.py --- @@ -33,10 +35,30 @@ def teardown_method(self): rmtree(self.path, ignore_errors=True)

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-07 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r91277019 --- Diff: tests/mock/context.py --- @@ -15,19 +15,15 @@ import pytest - from aria import application_model_storage

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-05 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90898173 --- Diff: aria/orchestrator/context/toolbelt.py --- @@ -33,7 +33,7 @@ def dependent_node_instances(self): :return: """

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-05 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90879741 --- Diff: aria/orchestrator/context/toolbelt.py --- @@ -33,7 +33,7 @@ def dependent_node_instances(self): :return: """

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-05 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90883621 --- Diff: tests/storage/__init__.py --- @@ -31,9 +32,10 @@ def teardown_method(self): rmtree(self.path, ignore_errors=True)

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-05 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90883153 --- Diff: tests/orchestrator/workflows/builtin/test_execute_operation.py --- @@ -13,16 +13,26 @@ # See the License for the specific language

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-05 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90882506 --- Diff: aria/storage/sql_mapi.py --- @@ -54,19 +53,47 @@ def get(self, entry_id, include=None, filters=None, locking=False, **kwargs):

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90781017 --- Diff: aria/storage/sql_mapi.py --- @@ -0,0 +1,361 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90779972 --- Diff: tests/orchestrator/workflows/core/test_engine.py --- @@ -233,12 +212,13 @@ def mock_workflow(ctx, graph): class

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90777213 --- Diff: aria/storage/sql_mapi.py --- @@ -0,0 +1,361 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90778725 --- Diff: aria/storage/models.py --- @@ -422,23 +531,55 @@ def validate_max_attempts(_, value, *args): SUCCESS, FAILED,

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90778979 --- Diff: aria/storage/structures.py --- @@ -27,281 +27,189 @@ * Model - abstract model implementation. """ import json

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90778697 --- Diff: aria/storage/models.py --- @@ -148,265 +302,220 @@ def __lt__(self, other): return False -class

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90778632 --- Diff: aria/storage/models.py --- @@ -148,265 +302,220 @@ def __lt__(self, other): return False -class

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90779218 --- Diff: aria/storage/structures.py --- @@ -27,281 +27,189 @@ * Model - abstract model implementation. """ import json

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90779014 --- Diff: aria/storage/structures.py --- @@ -27,281 +27,189 @@ * Model - abstract model implementation. """ import json

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90778672 --- Diff: aria/storage/models.py --- @@ -148,265 +302,220 @@ def __lt__(self, other): return False -class

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90779099 --- Diff: aria/storage/structures.py --- @@ -27,281 +27,189 @@ * Model - abstract model implementation. """ import json

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90778768 --- Diff: aria/storage/models.py --- @@ -422,23 +531,55 @@ def validate_max_attempts(_, value, *args): SUCCESS, FAILED,

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90779283 --- Diff: aria/storage/structures.py --- @@ -27,281 +27,189 @@ * Model - abstract model implementation. """ import json

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90779796 --- Diff: tests/mock/context.py --- @@ -13,21 +13,59 @@ # See the License for the specific language governing permissions and #

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90779347 --- Diff: aria/storage/structures.py --- @@ -27,281 +27,189 @@ * Model - abstract model implementation. """ import json

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90778551 --- Diff: aria/storage/models.py --- @@ -148,265 +302,220 @@ def __lt__(self, other): return False -class

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90780128 --- Diff: tests/storage/test_models.py --- @@ -187,111 +99,18 @@ def test_deployment_update_step_model(): assert remove_rel <

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90778657 --- Diff: aria/storage/models.py --- @@ -148,265 +302,220 @@ def __lt__(self, other): return False -class

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90779244 --- Diff: aria/storage/structures.py --- @@ -27,281 +27,189 @@ * Model - abstract model implementation. """ import json

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90778954 --- Diff: aria/storage/structures.py --- @@ -27,281 +27,189 @@ * Model - abstract model implementation. """ import json

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90779747 --- Diff: aria/storage/models.py --- @@ -59,66 +76,203 @@ 'Plugin', ) -# todo: sort this, maybe move from mgr or move from

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread ran-z
Github user ran-z commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90773942 --- Diff: aria/storage/sql_mapi.py --- @@ -0,0 +1,361 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread ran-z
Github user ran-z commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90774109 --- Diff: aria/storage/sql_mapi.py --- @@ -0,0 +1,361 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread ran-z
Github user ran-z commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90774280 --- Diff: aria/storage/sql_mapi.py --- @@ -0,0 +1,361 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread ran-z
Github user ran-z commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90774188 --- Diff: aria/storage/sql_mapi.py --- @@ -0,0 +1,361 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread ran-z
Github user ran-z commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90774557 --- Diff: aria/storage/sql_mapi.py --- @@ -0,0 +1,361 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread ran-z
Github user ran-z commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90773988 --- Diff: aria/storage/sql_mapi.py --- @@ -0,0 +1,361 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-04 Thread ran-z
Github user ran-z commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90774547 --- Diff: aria/storage/sql_mapi.py --- @@ -0,0 +1,361 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90503715 --- Diff: aria/storage/mapi/sql.py --- @@ -0,0 +1,368 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90504139 --- Diff: aria/storage/models.py --- @@ -60,353 +73,612 @@ ) # todo: sort this, maybe move from mgr or move from aria???

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90504471 --- Diff: aria/storage/models.py --- @@ -60,353 +73,612 @@ ) # todo: sort this, maybe move from mgr or move from aria???

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90503205 --- Diff: aria/storage/mapi/sql.py --- @@ -0,0 +1,368 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90504075 --- Diff: aria/storage/models.py --- @@ -60,353 +73,612 @@ ) # todo: sort this, maybe move from mgr or move from aria???

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90505762 --- Diff: aria/storage/models.py --- @@ -60,353 +73,612 @@ ) # todo: sort this, maybe move from mgr or move from aria???

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90502146 --- Diff: aria/storage/filesystem_api.py --- @@ -0,0 +1,39 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90501806 --- Diff: aria/storage/api.py --- @@ -0,0 +1,219 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90502764 --- Diff: aria/storage/mapi/inmemory.py --- @@ -0,0 +1,148 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more --- End

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90503314 --- Diff: aria/storage/mapi/sql.py --- @@ -0,0 +1,368 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90499710 --- Diff: aria/storage/__init__.py --- @@ -37,354 +37,93 @@ * drivers - module, a pool of Aria standard drivers. * StorageDriver

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90496245 --- Diff: aria/orchestrator/context/workflow.py --- @@ -64,19 +66,27 @@ def nodes(self): """ Iterator over nodes

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90501110 --- Diff: aria/storage/api.py --- @@ -0,0 +1,219 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90501812 --- Diff: aria/storage/api.py --- @@ -0,0 +1,219 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90498171 --- Diff: aria/storage/__init__.py --- @@ -37,354 +37,93 @@ * drivers - module, a pool of Aria standard drivers. * StorageDriver

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90495335 --- Diff: aria/orchestrator/context/workflow.py --- @@ -64,19 +66,27 @@ def nodes(self): """ Iterator over nodes

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90502574 --- Diff: aria/storage/mapi/filesystem.py --- @@ -0,0 +1,118 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more ---

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90499868 --- Diff: aria/storage/__init__.py --- @@ -37,354 +37,93 @@ * drivers - module, a pool of Aria standard drivers. * StorageDriver

[GitHub] incubator-ariatosca pull request #31: ARIA-30-SQL-based-storage-implementati...

2016-12-01 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/31#discussion_r90500929 --- Diff: aria/storage/api.py --- @@ -0,0 +1,219 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor

[GitHub] incubator-ariatosca pull request #31: Aria 30 sql based storage implementati...

2016-12-01 Thread mxmrlv
GitHub user mxmrlv opened a pull request: https://github.com/apache/incubator-ariatosca/pull/31 Aria 30 sql based storage implementation You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/incubator-ariatosca