Skip to content

Commit 7166b6f

Browse files
committed
Apply review comment
1 parent 0b4221e commit 7166b6f

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

proposals/POEM-support-array-result.md

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -18,38 +18,38 @@
1818
-->
1919
# Title
2020

21-
Currently, openwhisk supports return json object only, e.g.
21+
Currently, OpenWhisk supports returning a JSON object only, e.g.
2222
```shell
2323
# wsk action invoke hello -r
2424
{
2525
"greeting": "Hello stranger!"
2626
}
2727
```
28-
It is necessary to support return array for common action, e.g.
28+
It is necessary to support returning an array too as an array is also a proper JSON object, e.g.
2929
```shell
3030
# wsk action invoke hello-array -r
3131
[
3232
"a",
3333
"b"
3434
]
3535
```
36-
For sequence action, need to support as well.
36+
The sequence action should be considered support as well.
3737

3838
# Status
3939
* Current state: In-progress
4040
* Author(s): @ningyougang
4141

4242
# Summary and Motivation
4343

44-
This POEM proposes a new feature that allows user to write their own action which supports array result.
45-
So the result will support object and array both in future.
44+
This POEM proposes a new feature that allows user to write their own action which supports an array result.
45+
So actions would be able to return a JSON object or an array.
4646

4747
# Proposed changes
4848
## Openwhisk main repo
49-
Make controller and invoker support array result both.
49+
Make controller and invoker support both a JSON object and an array result.
5050

5151
## Runtime repos
52-
All runtime images should support array result. e.g.
52+
All runtime images should support an array result. e.g.
5353

5454
* nodejs (supports by default)
5555
* go
@@ -66,5 +66,5 @@ All runtime images should support array result. e.g.
6666
* ballerina
6767

6868
## Openwhisk-cli repo
69-
* When use wsk to execute action, need to support parse array result.
70-
* When use wsk to get activation, need to support parse array result as well.
69+
* The `wsk` CLI needs to support parsing an array result when executing actions.
70+
* The `wsk` CLI needs to support parsing an array result when getting activations.

0 commit comments

Comments
 (0)