voidmatcha commented on PR #5254: URL: https://github.com/apache/zeppelin/pull/5254#issuecomment-4609109426
@kevinjmh 로컬에서 직접 확인했고, 제가 #5044에서 했던 컴포넌트 우회 방식보다 이 방향이 더 맞다고 봅니다. 진짜 원인은 여기서 제거하신 `lastMsgIdSeqSent > msgIdSeqReceived` 필터입니다 — `COPY_PARAGRAPH` 요청과 응답 사이에 `EDITOR_SETTING` 같은 무관한 메시지가 끼면 정당한 `OP.PARAGRAPH` echo를 false-positive로 떨어뜨립니다. 필터를 없애고 중복은 `isUpdateRequired()`가 흡수하게 하는 게 깔끔합니다. **이 PR 머지되면 #5044는 닫겠습니다.** 검증: 클론 내용 보존, PENDING/RUNNING/FINISHED 전환도 즉시 동작(personalized 모드 포함). 원래 필터가 막던 다른 케이스 — 언급하신 미검증 부분이라 정리했습니다. 이 필터는 `msgId`가 있는 `OP.PARAGRAPH`에만 작동합니다(위의 `if (!message.msgId) return true` 때문). 영향 범위는 네 경로뿐입니다: - `COPY_PARAGRAPH`(클론): 수정 + 검증 완료 - personalized `RUN_PARAGRAPH` unicast: 검증 완료 - `COMMIT_PARAGRAPH` self-echo: 내용 일치라 `isUpdateRequired()`에서 no-op — spot-check 권장 - `PARAGRAPH_CLEAR_OUTPUT`: echo가 서버 상태 재적용뿐 — spot-check 권장 PENDING/RUNNING/FINISHED 라이프사이클은 `MSG_ID_NOT_DEFINED`라 애초에 이 필터를 안 거치므로 이 변경과 무관합니다. 후속 두 가지(이 PR 범위 밖): - **클론 커서 focus**: 이 PR은 내용 손실만 고치고 focus는 아닙니다. 갓 클론된 문단은 커서가 wrapper에 남습니다(`addParagraph`의 `focus = true`가 같은 tick에 생성된 에디터를 못 잡음). ZEPPELIN-6294로 별도 진행하겠습니다. - **dirty-text 덮어쓰기 노출 확대**: 필터가 사라지면 더 많은 echo가 `updateAllScopeTexts()`에 도달하는데, "keep local update" 분기가 주석과 달리 `paragraph.text`를 덮어써 편집 중 텍스트가 날아갈 수 있습니다. 기존부터 있던 문제고(필터가 가려줬을 뿐) 이 PR 범위는 아니지만, 별도 이슈로 추적하면 좋겠습니다. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
