ZEPPELIN-1364: Return only the ran paragraph details for indiviual paragraph run.

[vipul1409]

What is this PR for?

If you run a note book having lots of paragraphs and few images and svgs. Every paragraph run sends the complete notebook instead of sending the details of just the executed paragraph. We can see good performance gain if we send only the details of ran paragraph.

What type of PR is it?

[Improvement]

Todos

• - Task

What is the Jira issue?

• Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/
• Put link here, and add [ZEPPELIN-Jira number] in PR title, eg. [ZEPPELIN-533]

https://issues.apache.org/jira/browse/ZEPPELIN-1364

How should this be tested?

Create a notebook with few large images etc. so that note.json size is around 4 MB.
Run %sh ls as one paragraph.
Time take for paragraph to move to pending and finished status will be very high, as the complete note will be sent for each step. If we add this fix the status changes are very fast and smooth.

Screenshots (if appropriate)

Questions:

• Does the licenses files need update?
  No
• Is there breaking changes for older versions?
  No
• Does this needs documentation?
  No

[echarles]

Didn't try the changes but definitively +1 for this to enhance performance (tried to imagine side effects but didn't find any atm)

[bzz]

@vipul1409 thank you, it sounds interesting! Could you please let know, if it is ready for review?

@Leemoonsoo @corneadoug do you guys see any downsides\side-effects for such change?

Seems like CI is failing for some reason, could you please check it? Usually posting the reason of CI failure speeds up the review process a lot.

[vipul1409]

Yes It is ready for review.
The reason for CI failure looks unrelated to my changes as the build is failing in npm side and I have not made any JS changes.
The build is passing for few profiles and failing for few looks intermittent.
This is the error.

[ERROR] npm ERR! Callback called more than once.
[ERROR] npm ERR!

Any suggestion?

[corneadoug]

@vipul1409 Can you rebase this PR?
I will run a few tests to see if all the paragraph updates are handled here.

[vipul1409]

Not needed any more.