[ https://issues.apache.org/jira/browse/BEAM-8425?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ning Kang closed BEAM-8425. --------------------------- Fix Version/s: Not applicable Resolution: Abandoned We decide not to notify users but proactively steer the user away from the bad situation. By implementing show() that executes an implicitly built fragment, we mitigate the re-execution/deleted cell problems. > Notifying Interactive Beam user about Beam related cell deletion or > re-execution > -------------------------------------------------------------------------------- > > Key: BEAM-8425 > URL: https://issues.apache.org/jira/browse/BEAM-8425 > Project: Beam > Issue Type: New Feature > Components: runner-py-interactive > Reporter: Ning Kang > Assignee: Ning Kang > Priority: Major > Fix For: Not applicable > > > There is a general problem about Interactive Notebooks that when an end user > deletes a cell that has been executed or re-executes a cell, those previous > executions are hidden from the end user. > However, hidden states will still have side effects in the notebook. > This kind of problem bothers Beam even more because Beam's pipeline > construction statements are note idempotent and pipeline execution is > decoupled and deferred from pipeline construction. > Re-executing a cell with Beam statements that build a pipeline would cause > unexpected pipeline state and the user wouldn't notice it due to the problem > of notebooks. > We'll intercept each transform application invocation from the > InteractiveRunner and record the ipython/notebook prompt number. Then each > time a user executes a cell that applies PTransform, we'll compare the > recorded list of prompt numbers with current notebook file's content and > figure out if there is any missing number. If so, we know for sure that a > re-execution happens and use display manager to notify the end user of > potential side effects caused by hidden states of the notebook/ipython. -- This message was sent by Atlassian Jira (v8.3.4#803005)