You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@texera.apache.org by "Ma77Ball (via GitHub)" <gi...@apache.org> on 2026/07/21 03:31:56 UTC
[PR] test(workflow-core): cover isFileResolved and parse owner [texera]
Ma77Ball opened a new pull request, #6699:
URL: https://github.com/apache/texera/pull/6699
### What changes were proposed in this PR?
- Add unit tests for FileResolver.isFileResolved covering non-empty scheme, missing scheme, and malformed URI cases.
- Add unit tests for FileResolver.parseDatasetOwnerAndName covering valid paths, too few segments, and a null path.
### Any related issues, documentation, discussions?
Closes: #6698
### How was this PR tested?
- Run: `sbt "WorkflowCore/testOnly *FileResolverSpec"`, expect all 9 tests passing.
- Test-only change; no production code is modified.
### Was this PR authored or co-authored using generative AI tooling?
Co-authored with Claude Opus 4.8 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] test(workflow-core): cover isFileResolved and parse owner [texera]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #6699:
URL: https://github.com/apache/texera/pull/6699#issuecomment-5029869814
## [Codecov](https://app.codecov.io/gh/apache/texera/pull/6699?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 74.65%. Comparing base ([`0467c76`](https://app.codecov.io/gh/apache/texera/commit/0467c76529981b3951bceb33eedf6bcc776f16a3?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`2c9ebde`](https://app.codecov.io/gh/apache/texera/commit/2c9ebdec2a964138515fb9b953dc52fda9b1cae2?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 #6699 +/- ##
=========================================
Coverage 74.65% 74.65%
Complexity 3438 3438
=========================================
Files 1160 1160
Lines 45793 45793
Branches 5069 5069
=========================================
Hits 34189 34189
Misses 9953 9953
Partials 1651 1651
```
| [Flag](https://app.codecov.io/gh/apache/texera/pull/6699/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/6699/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `70.00% <ø> (ø)` | | |
| [agent-service](https://app.codecov.io/gh/apache/texera/pull/6699/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 [0467c76](https://app.codecov.io/gh/apache/texera/commit/0467c76529981b3951bceb33eedf6bcc776f16a3?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/6699/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `66.81% <ø> (ø)` | | Carriedforward from [0467c76](https://app.codecov.io/gh/apache/texera/commit/0467c76529981b3951bceb33eedf6bcc776f16a3?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/6699/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `18.00% <ø> (ø)` | | Carriedforward from [0467c76](https://app.codecov.io/gh/apache/texera/commit/0467c76529981b3951bceb33eedf6bcc776f16a3?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/6699/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `66.66% <ø> (ø)` | | |
| [file-service](https://app.codecov.io/gh/apache/texera/pull/6699/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `66.80% <ø> (ø)` | | Carriedforward from [0467c76](https://app.codecov.io/gh/apache/texera/commit/0467c76529981b3951bceb33eedf6bcc776f16a3?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/6699/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `78.56% <ø> (ø)` | | Carriedforward from [0467c76](https://app.codecov.io/gh/apache/texera/commit/0467c76529981b3951bceb33eedf6bcc776f16a3?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [notebook-migration-service](https://app.codecov.io/gh/apache/texera/pull/6699/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 [0467c76](https://app.codecov.io/gh/apache/texera/commit/0467c76529981b3951bceb33eedf6bcc776f16a3?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/6699/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `91.99% <ø> (ø)` | | Carriedforward from [0467c76](https://app.codecov.io/gh/apache/texera/commit/0467c76529981b3951bceb33eedf6bcc776f16a3?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/6699/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 [0467c76](https://app.codecov.io/gh/apache/texera/commit/0467c76529981b3951bceb33eedf6bcc776f16a3?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/6699?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
Re: [PR] test(workflow-core): cover isFileResolved and parse owner [texera]
Posted by "aglinxinyuan (via GitHub)" <gi...@apache.org>.
aglinxinyuan merged PR #6699:
URL: https://github.com/apache/texera/pull/6699
--
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] test(workflow-core): cover isFileResolved and parse owner [texera]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #6699:
URL: https://github.com/apache/texera/pull/6699#issuecomment-5029892278
<!-- texera-benchmarks-comment -->
## ⚠️ Benchmark changes need a look
🟢 0 better · 🔴 9 worse · ⚪ 6 noise (<±5%) · 0 without baseline
> Compared against main `0467c76` benchmarked on **this same runner**, so the delta is largely free of cross-runner hardware noise. The "7d avg" column still reflects the gh-pages dashboard. Treat <±5% as noise unless repeated.
[Dashboard](https://apache.github.io/texera/dev/bench/) · [Run](https://github.com/apache/texera/actions/runs/29798751624)
| | config | throughput | MB/s | latency | max Δ latest / 7d |
|---|---|---:|---:|---:|---:|
| 🔴 | bs=10 sw=10 sl=64 | 381 | 0.233 | 25,475/33,639/33,639 us | 🔴 -17.4% / 🔴 +115.2% |
| 🔴 | bs=100 sw=10 sl=64 | 942 | 0.575 | 105,755/125,540/125,540 us | 🔴 +8.9% / 🔴 +18.4% |
| 🔴 | bs=1000 sw=10 sl=64 | 1,093 | 0.667 | 904,121/1,030,623/1,030,623 us | 🔴 +5.9% / 🟢 -7.2% |
<details><summary>Baseline details</summary>
Latest main `0467c76` from same runner
| config | metric | PR | latest main | 7d avg | Δ latest | Δ 7d |
|---|---|---:|---:|---:|---:|---:|
| bs=10 sw=10 sl=64 | throughput | 381 tuples/sec | 461 tuples/sec | 767.23 tuples/sec | -17.4% | -50.3% |
| bs=10 sw=10 sl=64 | MB/s | 0.233 MB/s | 0.282 MB/s | 0.468 MB/s | -17.4% | -50.2% |
| bs=10 sw=10 sl=64 | p50 | 25,475 us | 21,708 us | 12,705 us | +17.4% | +100.5% |
| bs=10 sw=10 sl=64 | p95 | 33,639 us | 31,885 us | 15,633 us | +5.5% | +115.2% |
| bs=10 sw=10 sl=64 | p99 | 33,639 us | 31,885 us | 20,214 us | +5.5% | +66.4% |
| bs=100 sw=10 sl=64 | throughput | 942 tuples/sec | 967 tuples/sec | 1,001 tuples/sec | -2.6% | -5.9% |
| bs=100 sw=10 sl=64 | MB/s | 0.575 MB/s | 0.59 MB/s | 0.611 MB/s | -2.5% | -5.9% |
| bs=100 sw=10 sl=64 | p50 | 105,755 us | 104,301 us | 99,949 us | +1.4% | +5.8% |
| bs=100 sw=10 sl=64 | p95 | 125,540 us | 115,245 us | 105,996 us | +8.9% | +18.4% |
| bs=100 sw=10 sl=64 | p99 | 125,540 us | 115,245 us | 117,999 us | +8.9% | +6.4% |
| bs=1000 sw=10 sl=64 | throughput | 1,093 tuples/sec | 1,055 tuples/sec | 1,033 tuples/sec | +3.6% | +5.8% |
| bs=1000 sw=10 sl=64 | MB/s | 0.667 MB/s | 0.644 MB/s | 0.631 MB/s | +3.6% | +5.8% |
| bs=1000 sw=10 sl=64 | p50 | 904,121 us | 950,901 us | 974,571 us | -4.9% | -7.2% |
| bs=1000 sw=10 sl=64 | p95 | 1,030,623 us | 973,223 us | 1,016,467 us | +5.9% | +1.4% |
| bs=1000 sw=10 sl=64 | p99 | 1,030,623 us | 973,223 us | 1,048,798 us | +5.9% | -1.7% |
</details>
<details><summary>Raw CSV</summary>
```csv
config_idx,batch_size,schema_width,string_len,num_batches,total_ms,total_tuples,total_bytes,tuples_per_sec,mb_per_sec,lat_p50_us,lat_p95_us,lat_p99_us
0,10,10,64,20,524.36,200,128000,381,0.233,25475.17,33639.16,33639.16
1,100,10,64,20,2122.58,2000,1280000,942,0.575,105754.99,125539.68,125539.68
2,1000,10,64,20,18298.07,20000,12800000,1093,0.667,904121.24,1030622.51,1030622.51
```
</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
Re: [PR] test(workflow-core): cover isFileResolved and parse owner [texera]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #6699:
URL: https://github.com/apache/texera/pull/6699#issuecomment-5029853261
<!-- 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:** `@Yicong-Huang`
You can notify them by mentioning `@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