[ZEPPELIN-3713] Remove paragraph text and title from NotebookService.runParagraph

• Type: Bug
• Status: Open
• Priority: Critical
• Resolution: Unresolved
• Affects Version/s: 0.9.0
• Component/s: zeppelin-server
• Assignee: George Klimov
• Reporter: George Klimov
• Created: 14/Aug/18 17:19
• Updated: 17/Aug/18 09:10

Description

NotebookService.runParagraph has paragraph text and title as args, it seems to be unnecessary, because paragraphs extracted from the note already contains them. Also it might cause new bugs related to incorrect text/title passing.

For example, bug in NotebookRestApi.runParagraph():

@POST
@Path("job/{noteId}/{paragraphId}")
@ZeppelinApi
public Response runParagraph(@PathParam("noteId") String noteId,
@PathParam("paragraphId") String paragraphId, String message)
throws IOException, IllegalArgumentException {

...

notebookService.runParagraph(
    noteId, paragraphId, "", "", params,
    new HashMap<String, Object>(), false, getServiceContext(), new RestServiceCallback<>());
return new JsonResponse<>(Status.OK).build();
}

all paragraphs processing as blank.

Issue Links

• links to: GitHub Pull Request #3140