You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@texera.apache.org by "kz930 (via GitHub)" <gi...@apache.org> on 2026/10/20 22:34:42 UTC
[PR] feat(frontend): keep only the console in the bottom panel [texera]
kz930 opened a new pull request, #9060:
URL: https://github.com/apache/texera/pull/9060
### What changes were proposed in this PR?
With results open beside their operators (#9058), the bottom panel drops its Result tab and keeps only the console.
With no operator selected, the console shows every operator's messages in time order, each tagged with its operator; clicking a tag narrows it to that operator, as selecting the operator does. With one operator selected, it shows that operator's messages alone, under a line naming it that closes back to all. The errors from the Static Error tab join the console as error messages, shown by the rule the tab used: an operator's own errors with it, and errors that belong to no operator when none is selected. That tab and its component are removed.
(screenshots)
This needs #9058 and #9056, so the diff includes their commits, and #9054's, until they merge. The PR's own change is the last commit.
### Any related issues, documentation, discussions?
Closes #9059. Part of #9052. Design discussed in #8842.
### How was this PR tested?
The specs of the console, the console service and the result panel cover every operator's messages and their tags, narrowing to one operator and back, errors as console messages, and a re-render with an error present not moving the list. The full frontend suite passes. In the running app, two Python UDFs' prints and an error appear together, tagged by operator, and narrow to one operator when it is selected.
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Claude Opus 5.5)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(frontend): keep only the console in the bottom panel [texera]
Posted by "kz930 (via GitHub)" <gi...@apache.org>.
kz930 commented on PR #9060:
URL: https://github.com/apache/texera/pull/9060#issuecomment-6304453651
@carloea2 heads-up: this PR was reorganized to split the work more evenly. It now holds only the console change; the pinned windows moved to #9063. Its own change is still the last commit.
--
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: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(frontend): keep only the console in the bottom panel [texera]
Posted by "kz930 (via GitHub)" <gi...@apache.org>.
kz930 commented on PR #9060:
URL: https://github.com/apache/texera/pull/9060#issuecomment-6303485706
@carloea2 could you take a look at this when you have a chance? It stays a draft until #9058 and #9056 merge, but its own change, the last commit, is ready for review.
--
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: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(frontend): keep only the console in the bottom panel [texera]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #9060:
URL: https://github.com/apache/texera/pull/9060#issuecomment-6304686873
<!-- texera-reviewer-suggestion -->
### Automated Reviewer Suggestions
Based on the `git blame` history of the changed files, we recommend the following reviewers:
- **Contributors with relevant context:** `@yangzhang75`, `@aglinxinyuan`, `@sshiv012`
You can notify them by mentioning `@yangzhang75`, `@aglinxinyuan`, `@sshiv012` in a comment.
--
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: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(frontend): keep only the console in the bottom panel [texera]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #9060:
URL: https://github.com/apache/texera/pull/9060#issuecomment-6305875021
## [Codecov](https://app.codecov.io/gh/apache/texera/pull/9060?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:x: Patch coverage is `71.07062%` with `127 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 92.17%. Comparing base ([`22b82a2`](https://app.codecov.io/gh/apache/texera/commit/22b82a2958f8328be2f05044092e2b30e38c56cb?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`d127081`](https://app.codecov.io/gh/apache/texera/commit/d12708147b9c58b1830145ab5b681a98da619c2d?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/texera/pull/9060?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...onent/workflow-editor/workflow-editor.component.ts](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fcomponent%2Fworkflow-editor%2Fworkflow-editor.component.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2UvY29tcG9uZW50L3dvcmtmbG93LWVkaXRvci93b3JrZmxvdy1lZGl0b3IuY29tcG9uZW50LnRz) | 48.10% | [36 Missing and 5 partials :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...-result-window/pinned-result-window.component.html](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fcomponent%2Fworkflow-editor%2Fpinned-result-window%2Fpinned-result-window.component.html&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2UvY29tcG9uZW50L3dvcmtmbG93LWVkaXRvci9waW5uZWQtcmVzdWx0LXdpbmRvdy9waW5uZWQtcmVzdWx0LXdpbmRvdy5jb21wb25lbnQuaHRtbA==) | 0.00% | [29 Missing :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...ed-result-window/pinned-result-window.component.ts](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fcomponent%2Fworkflow-editor%2Fpinned-result-window%2Fpinned-result-window.component.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2UvY29tcG9uZW50L3dvcmtmbG93LWVkaXRvci9waW5uZWQtcmVzdWx0LXdpbmRvdy9waW5uZWQtcmVzdWx0LXdpbmRvdy5jb21wb25lbnQudHM=) | 52.08% | [18 Missing and 5 partials :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...ent/workflow-editor/workflow-editor.component.html](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fcomponent%2Fworkflow-editor%2Fworkflow-editor.component.html&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2UvY29tcG9uZW50L3dvcmtmbG93LWVkaXRvci93b3JrZmxvdy1lZGl0b3IuY29tcG9uZW50Lmh0bWw=) | 44.44% | [10 Missing :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...esult-preview/operator-result-preview.component.ts](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fcomponent%2Fworkflow-editor%2Foperator-result-preview%2Foperator-result-preview.component.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2UvY29tcG9uZW50L3dvcmtmbG93LWVkaXRvci9vcGVyYXRvci1yZXN1bHQtcHJldmlldy9vcGVyYXRvci1yZXN1bHQtcHJldmlldy5jb21wb25lbnQudHM=) | 89.39% | [0 Missing and 7 partials :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...ult-panel/console-frame/console-frame.component.ts](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fcomponent%2Fresult-panel%2Fconsole-frame%2Fconsole-frame.component.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2UvY29tcG9uZW50L3Jlc3VsdC1wYW5lbC9jb25zb2xlLWZyYW1lL2NvbnNvbGUtZnJhbWUuY29tcG9uZW50LnRz) | 85.36% | [1 Missing and 5 partials :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...result-table-frame/result-table-frame.component.ts](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fcomponent%2Fresult-panel%2Fresult-table-frame%2Fresult-table-frame.component.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2UvY29tcG9uZW50L3Jlc3VsdC1wYW5lbC9yZXN1bHQtdGFibGUtZnJhbWUvcmVzdWx0LXRhYmxlLWZyYW1lLmNvbXBvbmVudC50cw==) | 91.30% | [0 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...c/app/workspace/component/menu/menu.component.html](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fcomponent%2Fmenu%2Fmenu.component.html&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2UvY29tcG9uZW50L21lbnUvbWVudS5jb21wb25lbnQuaHRtbA==) | 50.00% | [2 Missing :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...ult-preview/operator-result-preview.component.html](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fcomponent%2Fworkflow-editor%2Foperator-result-preview%2Foperator-result-preview.component.html&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2UvY29tcG9uZW50L3dvcmtmbG93LWVkaXRvci9vcGVyYXRvci1yZXN1bHQtcHJldmlldy9vcGVyYXRvci1yZXN1bHQtcHJldmlldy5jb21wb25lbnQuaHRtbA==) | 95.34% | [1 Missing and 1 partial :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...ervice/workflow-graph/model/joint-graph-wrapper.ts](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fservice%2Fworkflow-graph%2Fmodel%2Fjoint-graph-wrapper.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2Uvc2VydmljZS93b3JrZmxvdy1ncmFwaC9tb2RlbC9qb2ludC1ncmFwaC13cmFwcGVyLnRz) | 60.00% | [2 Missing :warning: ](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| ... and [1 more](https://app.codecov.io/gh/apache/texera/pull/9060?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## main #9060 +/- ##
============================================
- Coverage 92.33% 92.17% -0.17%
+ Complexity 5045 5044 -1
============================================
Files 1261 1263 +2
Lines 54120 54351 +231
Branches 6768 6808 +40
============================================
+ Hits 49974 50097 +123
- Misses 2486 2566 +80
- Partials 1660 1688 +28
```
| [Flag](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | *Carryforward flag |
|---|---|---|---|
| [access-control-service](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `77.38% <ø> (ø)` | | Carriedforward from [a64630b](https://app.codecov.io/gh/apache/texera/commit/a64630b3afcfcfe57f8355445cb4949be2ab8346?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [agent-service](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `99.16% <ø> (ø)` | | Carriedforward from [a64630b](https://app.codecov.io/gh/apache/texera/commit/a64630b3afcfcfe57f8355445cb4949be2ab8346?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [amber](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `87.31% <ø> (-0.01%)` | :arrow_down: | Carriedforward from [a64630b](https://app.codecov.io/gh/apache/texera/commit/a64630b3afcfcfe57f8355445cb4949be2ab8346?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [computing-unit-managing-service](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `60.48% <ø> (ø)` | | Carriedforward from [a64630b](https://app.codecov.io/gh/apache/texera/commit/a64630b3afcfcfe57f8355445cb4949be2ab8346?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [config-service](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `87.37% <ø> (ø)` | | Carriedforward from [a64630b](https://app.codecov.io/gh/apache/texera/commit/a64630b3afcfcfe57f8355445cb4949be2ab8346?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [file-service](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `82.00% <ø> (ø)` | | Carriedforward from [a64630b](https://app.codecov.io/gh/apache/texera/commit/a64630b3afcfcfe57f8355445cb4949be2ab8346?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [frontend](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `96.15% <71.07%> (-0.41%)` | :arrow_down: | |
| [notebook-migration-service](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `83.73% <ø> (ø)` | | Carriedforward from [a64630b](https://app.codecov.io/gh/apache/texera/commit/a64630b3afcfcfe57f8355445cb4949be2ab8346?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [pyamber](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `98.57% <ø> (ø)` | | Carriedforward from [a64630b](https://app.codecov.io/gh/apache/texera/commit/a64630b3afcfcfe57f8355445cb4949be2ab8346?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [workflow-compiling-service](https://app.codecov.io/gh/apache/texera/pull/9060/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `72.43% <ø> (ø)` | | Carriedforward from [a64630b](https://app.codecov.io/gh/apache/texera/commit/a64630b3afcfcfe57f8355445cb4949be2ab8346?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
*This pull request uses carry forward flags. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to find out more.
</details>
[:umbrella: View full report in Codecov by Harness](https://app.codecov.io/gh/apache/texera/pull/9060?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
<details><summary> :rocket: New features to boost your workflow: </summary>
- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
--
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: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org