Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188989671
I don't have strong opinion here for adding parameter vs new api.
But i want to mention that adding parameter make the api return two
different things.
When sync is false, return success or fail for the job submission.
When sync is true, return success or fail for the job execution, with
result data.
That makes the api difficult to understand. And later, there could be some
additional parameter introduced for only for async api or only for sync api. in
that case, combination of different parameters need to be explained for users.
So i'm little bit biased to new api for this reasons. what do you guys
think?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
