You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@texera.apache.org by "PG1204 (via GitHub)" <gi...@apache.org> on 2026/07/27 05:41:23 UTC
[PR] refactor(frontend): make redundant operator border repaints a no-op [texera]
PG1204 opened a new pull request, #6927:
URL: https://github.com/apache/texera/pull/6927
### What changes were proposed in this PR?
When an operator is added, its border was painted by two paths - the operator-add restore and the validation pass, producing the same color. Harmless, but a redundant repaint.
This PR adds a guarded border setter (`paintOperatorBorder`) in `JointUIService` that writes `rect.body/stroke` only when the color actually changes. Both `changeOperatorColor` and `changeOperatorState` route their border write through it, so a repaint with the color the border already has becomes a no-op, effectively "painted once" - including on the navigation-return (reload) path.
**Deviation from the approach suggested on the issue:** the issue suggested dropping the `applyOperatorBorder` call from the operator-add handler and letting the validation pass set the border. I kept that call and used the guard instead, because `changeOperatorStatistics` already paints the border via `changeOperatorState` *without* checking validity. Dropping `applyOperatorBorder` would make an invalid operator with a cached "completed" status rely on the validation pass firing afterward to correct green→red, reintroducing the order-dependent border fragility that #5146 removed (and it would break in the edge case where `setDynamicSchema` skips its emit because the schema is unchanged). The guard reaches the same no-redundant-repaint goal while keeping the border validity-correct regardless of event timing.
### Any related issues, documentation, discussions?
Closes #5726
### How was this PR tested?
Unit tests:
- `JointUIService`: the guarded setter skips the write when the border is already the requested color, and writes when it differs.
- `WorkflowEditorComponent`: added a navigation-return test for a cached **Running** operator (orange), alongside the existing completed (green), default (gray), invalid (red), and invalid-over-cached-priority cases.
- Full frontend suite: 3739 passing.
Manual (navigated away from a running workflow and back), border restored correctly for:
- Completed operators → green
- Running operators → orange
- Invalid operators → red
- Valid, not run → default gray
### Was this PR authored or co-authored using generative AI tooling?
This PR was co-authored using Claude Code (Anthropic Claude Opus 4.7) in compliance with ASF.
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "PG1204 (via GitHub)" <gi...@apache.org>.
PG1204 commented on code in PR #6927:
URL: https://github.com/apache/texera/pull/6927#discussion_r3699702916
##########
frontend/src/app/workspace/service/joint-ui/joint-ui.service.ts:
##########
@@ -460,10 +460,12 @@ export class JointUIService {
}
jointPaper.getModelById(operatorID).attr({
[`.${operatorStateClass}`]: { text: operatorState.toString(), fill: fillColor },
- "rect.body": { stroke: fillColor },
[`.${operatorPortMetricsClass}`]: { fill: fillColor },
[`.${operatorWorkerCountClass}`]: { fill: fillColor },
});
+ // Border stroke goes through the guarded setter so a repaint with the color
+ // it already has is a no-op (see #5726).
+ this.paintOperatorBorder(jointPaper, operatorID, fillColor);
Review Comment:
addressed in [c3d7393](https://github.com/apache/texera/pull/6927/commits/c3d7393c01c5dc1d97d25dd136194d3ff6cd655c)
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "Xiao-zhen-Liu (via GitHub)" <gi...@apache.org>.
Xiao-zhen-Liu merged PR #6927:
URL: https://github.com/apache/texera/pull/6927
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "PG1204 (via GitHub)" <gi...@apache.org>.
PG1204 commented on code in PR #6927:
URL: https://github.com/apache/texera/pull/6927#discussion_r3699704323
##########
frontend/src/app/workspace/service/joint-ui/joint-ui.service.ts:
##########
@@ -491,11 +493,22 @@ export class JointUIService {
* @param isOperatorValid
*/
public changeOperatorColor(jointPaper: joint.dia.Paper, operatorID: string, isOperatorValid: boolean): void {
- if (isOperatorValid) {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "#CFCFCF");
- } else {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "red");
+ this.paintOperatorBorder(jointPaper, operatorID, isOperatorValid ? "#CFCFCF" : "red");
+ }
+
+ /**
+ * Sets the operator's border stroke, skipping the write when the border is
+ * already that color. On operator add, the operator-add restore and the
+ * validation pass both request a border color for the same operator; guarding
+ * the write here makes the redundant repaint a no-op (see #5726) and likewise
Review Comment:
addressed in [c3d7393](https://github.com/apache/texera/pull/6927/commits/c3d7393c01c5dc1d97d25dd136194d3ff6cd655c)
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #6927:
URL: https://github.com/apache/texera/pull/6927#issuecomment-5087675154
<!-- 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:** `@mengw15`, `@Ma77Ball`, `@Yicong-Huang`
You can notify them by mentioning `@mengw15`, `@Ma77Ball`, `@Yicong-Huang` 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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "PG1204 (via GitHub)" <gi...@apache.org>.
PG1204 commented on code in PR #6927:
URL: https://github.com/apache/texera/pull/6927#discussion_r3731039549
##########
frontend/src/app/workspace/service/joint-ui/joint-ui.service.ts:
##########
@@ -491,11 +491,24 @@ export class JointUIService {
* @param isOperatorValid
*/
public changeOperatorColor(jointPaper: joint.dia.Paper, operatorID: string, isOperatorValid: boolean): void {
- if (isOperatorValid) {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "#CFCFCF");
- } else {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "red");
+ this.paintOperatorBorder(jointPaper, operatorID, isOperatorValid ? "#CFCFCF" : "red");
+ }
+
+ /**
+ * Sets the operator's border stroke, returning early when it is already that
+ * color. A same-value attr() write would not re-render (Backbone's Model.set
+ * no-ops via _.isEqual), but attr() still deep-clones and deep-compares the
+ * whole attrs tree before reaching that check (~46us on our elements, down to
Review Comment:
done, comment modified - numbers were dropped and mechanism was kept.
[8e8c87c](https://github.com/apache/texera/pull/6927/commits/8e8c87cc5593469cc30c5ab7c640c38a483f9c1a)
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "PG1204 (via GitHub)" <gi...@apache.org>.
PG1204 commented on code in PR #6927:
URL: https://github.com/apache/texera/pull/6927#discussion_r3699711279
##########
frontend/src/app/workspace/component/workflow-editor/workflow-editor.component.spec.ts:
##########
@@ -911,6 +912,18 @@ describe("WorkflowEditorComponent", () => {
expect(getStroke(mockScanPredicate.operatorID)).toBe("green");
});
+ it("paints the execution-state stroke (orange) for a valid operator with a cached Running status", () => {
Review Comment:
addressed in [c3d7393](https://github.com/apache/texera/pull/6927/commits/c3d7393c01c5dc1d97d25dd136194d3ff6cd655c)
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "PG1204 (via GitHub)" <gi...@apache.org>.
PG1204 commented on code in PR #6927:
URL: https://github.com/apache/texera/pull/6927#discussion_r3699714426
##########
frontend/src/app/workspace/component/workflow-editor/workflow-editor.component.spec.ts:
##########
@@ -911,6 +912,18 @@ describe("WorkflowEditorComponent", () => {
expect(getStroke(mockScanPredicate.operatorID)).toBe("green");
});
+ it("paints the execution-state stroke (orange) for a valid operator with a cached Running status", () => {
Review Comment:
i have changed the pr description to Part of https://github.com/apache/texera/issues/5726 for now
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "PG1204 (via GitHub)" <gi...@apache.org>.
PG1204 commented on PR #6927:
URL: https://github.com/apache/texera/pull/6927#issuecomment-5087685095
/request-review @Xiao-zhen-Liu
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "PG1204 (via GitHub)" <gi...@apache.org>.
PG1204 commented on code in PR #6927:
URL: https://github.com/apache/texera/pull/6927#discussion_r3731039549
##########
frontend/src/app/workspace/service/joint-ui/joint-ui.service.ts:
##########
@@ -491,11 +491,24 @@ export class JointUIService {
* @param isOperatorValid
*/
public changeOperatorColor(jointPaper: joint.dia.Paper, operatorID: string, isOperatorValid: boolean): void {
- if (isOperatorValid) {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "#CFCFCF");
- } else {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "red");
+ this.paintOperatorBorder(jointPaper, operatorID, isOperatorValid ? "#CFCFCF" : "red");
+ }
+
+ /**
+ * Sets the operator's border stroke, returning early when it is already that
+ * color. A same-value attr() write would not re-render (Backbone's Model.set
+ * no-ops via _.isEqual), but attr() still deep-clones and deep-compares the
+ * whole attrs tree before reaching that check (~46us on our elements, down to
Review Comment:
done, comment modified - numbers were dropped and mechanism was kept.
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "Xiao-zhen-Liu (via GitHub)" <gi...@apache.org>.
Xiao-zhen-Liu commented on code in PR #6927:
URL: https://github.com/apache/texera/pull/6927#discussion_r3726514613
##########
frontend/src/app/workspace/service/joint-ui/joint-ui.service.ts:
##########
@@ -491,11 +491,24 @@ export class JointUIService {
* @param isOperatorValid
*/
public changeOperatorColor(jointPaper: joint.dia.Paper, operatorID: string, isOperatorValid: boolean): void {
- if (isOperatorValid) {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "#CFCFCF");
- } else {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "red");
+ this.paintOperatorBorder(jointPaper, operatorID, isOperatorValid ? "#CFCFCF" : "red");
+ }
+
+ /**
+ * Sets the operator's border stroke, returning early when it is already that
+ * color. A same-value attr() write would not re-render (Backbone's Model.set
+ * no-ops via _.isEqual), but attr() still deep-clones and deep-compares the
+ * whole attrs tree before reaching that check (~46us on our elements, down to
Review Comment:
Optional, and this is my number being over-quoted rather than your mistake.
`~46us` / `~0.6us` / `roughly 99%` came from a synthetic node benchmark I ran against a reconstructed 25-selector attrs tree, not from a profiler on the running app. The ratio holds up, but the absolute microseconds will drift with element shape and JS engine, so a specific figure in a source comment reads more precise than the evidence behind it.
I'd either drop the numbers and keep the mechanism ("attr() deep-clones and deep-compares the whole attrs tree before reaching that check, which this skips"), or keep them and say they're a rough local measurement. Either way, fine to merge as is.
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "Xiao-zhen-Liu (via GitHub)" <gi...@apache.org>.
Xiao-zhen-Liu commented on code in PR #6927:
URL: https://github.com/apache/texera/pull/6927#discussion_r3696927267
##########
frontend/src/app/workspace/service/joint-ui/joint-ui.service.ts:
##########
@@ -491,11 +493,22 @@ export class JointUIService {
* @param isOperatorValid
*/
public changeOperatorColor(jointPaper: joint.dia.Paper, operatorID: string, isOperatorValid: boolean): void {
- if (isOperatorValid) {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "#CFCFCF");
- } else {
- jointPaper.getModelById(operatorID).attr("rect.body/stroke", "red");
+ this.paintOperatorBorder(jointPaper, operatorID, isOperatorValid ? "#CFCFCF" : "red");
+ }
+
+ /**
+ * Sets the operator's border stroke, skipping the write when the border is
+ * already that color. On operator add, the operator-add restore and the
+ * validation pass both request a border color for the same operator; guarding
+ * the write here makes the redundant repaint a no-op (see #5726) and likewise
Review Comment:
The redundant repaint was already a no-op. Backbone's `Model.set` compares with `_.isEqual` and fires no change event for a same-value write, so `attr("rect.body/stroke", sameColor)` never reached the renderer.
What the guard really skips is jointjs cloning and deep-comparing the attrs tree, which is most of the cost of a redundant `attr()` call (~46us on our element, down to ~0.6us). That's the actual reason to keep it on `changeOperatorColor`, so it's worth saying that here instead.
##########
frontend/src/app/workspace/component/workflow-editor/workflow-editor.component.spec.ts:
##########
@@ -911,6 +912,18 @@ describe("WorkflowEditorComponent", () => {
expect(getStroke(mockScanPredicate.operatorID)).toBe("green");
});
+ it("paints the execution-state stroke (orange) for a valid operator with a cached Running status", () => {
Review Comment:
This is the part I'd like to keep. It's real coverage of the #3614 reload behaviour for a cached state other than Completed, independent of the rest of the PR.
One change: point the comment at #3614 and drop the #5726 reference, since I'm re-scoping that issue.
##########
frontend/src/app/workspace/service/joint-ui/joint-ui.service.ts:
##########
@@ -460,10 +460,12 @@ export class JointUIService {
}
jointPaper.getModelById(operatorID).attr({
[`.${operatorStateClass}`]: { text: operatorState.toString(), fill: fillColor },
- "rect.body": { stroke: fillColor },
[`.${operatorPortMetricsClass}`]: { fill: fillColor },
[`.${operatorWorkerCountClass}`]: { fill: fillColor },
});
+ // Border stroke goes through the guarded setter so a repaint with the color
+ // it already has is a no-op (see #5726).
+ this.paintOperatorBorder(jointPaper, operatorID, fillColor);
Review Comment:
This is the one spot where the guard can't pay off. The bundled `attr({...})` above already deep-merges and compares the whole attrs tree, so pulling the stroke out of it adds a second write rather than avoiding one.
Measured on the pinned versions, a real state change goes from ~45us to ~97us, and from one change event to two. Unchanged ticks are a wash.
Putting `"rect.body": { stroke: fillColor }` back in the bundled payload and dropping this call fixes it.
--
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] refactor(frontend): make redundant operator border repaints a no-op [texera]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #6927:
URL: https://github.com/apache/texera/pull/6927#issuecomment-5087728777
## [Codecov](https://app.codecov.io/gh/apache/texera/pull/6927?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 78.95%. Comparing base ([`49f9e2c`](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`5cb206a`](https://app.codecov.io/gh/apache/texera/commit/5cb206a8e6e6f14b88c254032d5835e8ca36ebe2?dropdown=coverage&el=desc&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 #6927 +/- ##
=========================================
Coverage 78.95% 78.95%
Complexity 3795 3795
=========================================
Files 1161 1161
Lines 46084 46087 +3
Branches 5110 5111 +1
=========================================
+ Hits 36384 36387 +3
Misses 8077 8077
Partials 1623 1623
```
| [Flag](https://app.codecov.io/gh/apache/texera/pull/6927/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/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `70.00% <ø> (ø)` | | Carriedforward from [49f9e2c](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?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/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `76.76% <ø> (ø)` | | Carriedforward from [49f9e2c](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?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/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `72.20% <ø> (ø)` | | Carriedforward from [49f9e2c](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?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/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `20.49% <ø> (ø)` | | Carriedforward from [49f9e2c](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?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/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `66.66% <ø> (ø)` | | Carriedforward from [49f9e2c](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?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/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `67.21% <ø> (ø)` | | Carriedforward from [49f9e2c](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?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/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `82.91% <100.00%> (+<0.01%)` | :arrow_up: | |
| [notebook-migration-service](https://app.codecov.io/gh/apache/texera/pull/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `78.94% <ø> (ø)` | | Carriedforward from [49f9e2c](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?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/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `95.38% <ø> (ø)` | | Carriedforward from [49f9e2c](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?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/6927/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `55.14% <ø> (ø)` | | Carriedforward from [49f9e2c](https://app.codecov.io/gh/apache/texera/commit/49f9e2c414b116bf328bdb9154cc97dc4d1e7e9e?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/6927?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