Repository navigation
Add an opt-in Iron Head Spark SQL panel adapter - #1
Draft
boltonidze wants to merge 7 commits into
Draft
boltonidze wants to merge 7 commits into
boltonidze wants to merge 7 commits into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds an optional Spark SQL assistant beside the existing notebook editor. The iframe uses the independent Iron Head
/embed/sqlpanel and a versioned, origin/window/context-checked message bridge. SQL is inserted as a new paragraph through the normal notebook API and its ACL checks; existing text is never sent to the panel or replaced, and no paragraph is executed.Runtime configuration defaults to disabled. Insertion requires a unique bound Spark SQL interpreter, or an explicit notebook default that unambiguously selects one. Offline/read-only/ambiguous cases retain copying; duplicate insertion requests and stale notebook contexts are rejected. An iframe stays hidden until the expected handshake, so an incompatible stage release cannot display the ordinary executing chat in the panel.
Validation: 10 host boundary tests pass. Angular production compilation passes. The final local asset-copy step hit ENOSPC; the pilot WAR combines the two newly compiled bundles with byte-verified unchanged bundles/assets from the prior successful production build. ZIP and index/runtime references were verified. Server/interpreter JARs are unchanged. The image is pinned in the separate datasci configuration PR. Full browser insertion and ACL results are recorded in the prototype evaluation report.
This PR is based on the 0.12.1 migration branch and contains only the AI adapter, its tests and UI-only build helper. It does not modify the migration branch itself.
Related prototype PRs: Iron Head API/panel, Zeppelin adapter, balloon Helm wiring.
RAT now passes locally after adding the missing Apache header to inherited
joom/PORTS.md(the only initial license-check failure). Full upstream interpreter/build workflows are still separate from the targeted prototype checks.