You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@texera.apache.org by "xuang7 (via GitHub)" <gi...@apache.org> on 2026/07/27 23:13:51 UTC
[PR] feat(dataset): add contributor metadata storage and API [texera]
xuang7 opened a new pull request, #6952:
URL: https://github.com/apache/texera/pull/6952
<!--
Thanks for sending a pull request (PR)! Here are some tips for you:
1. If this is your first time, please read our contributor guidelines:
[Contributing to Texera](https://github.com/apache/texera/blob/main/CONTRIBUTING.md)
2. Ensure you have added or run the appropriate tests for your PR
3. If the PR is work in progress, mark it a draft on GitHub.
4. Please write your PR title to summarize what this PR proposes, we
are following Conventional Commits style for PR titles as well.
5. Be sure to keep the PR description updated to reflect all changes.
-->
### What changes were proposed in this PR?
<!--
Please clarify what changes you are proposing. The purpose of this section
is to outline the changes. Here are some tips for you:
1. If you propose a new API, clarify the use case for a new API.
2. If you fix a bug, you can clarify why it is a bug.
3. If it is a refactoring, clarify what has been changed.
3. It would be helpful to include a before-and-after comparison using
screenshots or GIFs.
4. Please consider writing useful notes for better and faster reviews.
-->
This PR adds contributor metadata to datasets so that dataset authors and other contributors can be properly acknowledged. Each dataset can record a list of contributors, where each contributor has a name, a creator flag, an email address, an affiliation, and a free-text comments field.
Changes:
- Adds a new `dataset_contributor` table with a foreign key to `dataset` and cascade deletion in `texera_ddl.sql`, along with the incremental migration script `sql/updates/29.sql` and changelog entry 29.
- Updates `file-service` so that contributors are:
- returned with the dataset detail response,
- accepted during dataset creation, and
- replaceable through a new `POST /dataset/{did}/update/contributors` endpoint protected by the existing WRITE-access check.
- Validates contributor names as required fields with a maximum length of 256 characters.
Currently, contributors are stored independently for each dataset. Cross-dataset contributor identity is left as a future extension. The UI will be added in a follow-up PR.
### Any related issues, documentation, discussions?
<!--
Please use this section to link other resources if not mentioned already.
1. If this PR fixes an issue, please include `Fixes #1234`, `Resolves #1234`
or `Closes #1234`. If it is only related, simply mention the issue number.
2. If there is design documentation, please add the link.
3. If there is a discussion in the mailing list, please add the link.
-->
Related to #6926
### How was this PR tested?
<!--
If tests were added, say they were added here. Or simply mention that if the PR
is tested with existing test cases. Make sure to include/update test cases that
check the changes thoroughly including negative and positive cases if possible.
If it was tested in a way different from regular unit tests, please clarify how
you tested step by step, ideally copy and paste-able, so that other reviewers can
test and check, and descendants can verify in the future. If tests were not added,
please describe why they were not added and/or why it was difficult to add.
-->
7 ScalaTest cases were added.
### Was this PR authored or co-authored using generative AI tooling?
<!--
If generative AI tooling has been used in the process of authoring this PR,
please include the phrase: 'Generated-by: ' followed by the name of the tool
and its version. If no, write 'No'.
Please refer to the [ASF Generative Tooling Guidance](https://www.apache.org/legal/generative-tooling.html) for details.
-->
Generated-by: Claude Code (Claude Fable 5)
--
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(dataset): add contributor metadata storage and API [texera]
Posted by "Copilot (via GitHub)" <gi...@apache.org>.
Copilot commented on code in PR #6952:
URL: https://github.com/apache/texera/pull/6952#discussion_r3678498571
##########
sql/updates/29.sql:
##########
@@ -0,0 +1,38 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+\c texera_db
+
+SET search_path TO texera_db;
+
+BEGIN;
+
+CREATE TABLE IF NOT EXISTS dataset_contributor
+(
+ cid SERIAL PRIMARY KEY,
+ did INT NOT NULL,
+ name VARCHAR(256) NOT NULL,
+ creator BOOLEAN NOT NULL DEFAULT FALSE,
+ email VARCHAR(256),
+ affiliation VARCHAR(256),
+ comments TEXT,
+ FOREIGN KEY (did) REFERENCES dataset(did) ON DELETE CASCADE
Review Comment:
The linked issue (#6926) calls out a distinct contributor **Role** field, but the new schema/API uses a free-text `comments` field instead (no `role` column/field). If the intent is to satisfy the issue’s contract, consider adding a dedicated `role` column (and corresponding API field) or renaming `comments` to `role` and updating any docs/UI plans accordingly.
##########
file-service/src/main/scala/org/apache/texera/service/resource/DatasetResource.scala:
##########
@@ -175,12 +176,74 @@ object DatasetResource {
normalized
}
+ /**
+ * Helper function to get the contributors using the did
+ */
+ def getContributorsByDid(ctx: DSLContext, did: Integer): List[Contributor] = {
+ ctx
+ .selectFrom(DATASET_CONTRIBUTOR)
+ .where(DATASET_CONTRIBUTOR.DID.eq(did))
+ .fetch()
+ .asScala
+ .toList
+ .map { record =>
+ Contributor(
+ name = record.getName,
+ creator = record.getCreator,
+ affiliation = record.getAffiliation,
+ email = record.getEmail,
+ comments = record.getComments
+ )
+ }
+ }
+
+ /**
+ * Helper function to insert the contributors of a dataset in one batch
+ */
+ def insertContributors(ctx: DSLContext, did: Integer, contributors: List[Contributor]): Unit = {
+ if (contributors.isEmpty) {
+ return
+ }
+ val records = contributors.map { contributor =>
+ if (contributor == null || contributor.name == null || contributor.name.trim.isEmpty) {
+ throw new BadRequestException("Each contributor must have a name")
+ }
+ if (
+ contributor.name.length > 256 ||
+ Option(contributor.email).exists(_.length > 256) ||
+ Option(contributor.affiliation).exists(_.length > 256)
+ ) {
+ throw new BadRequestException("Contributor fields must not exceed 256 characters")
+ }
+ val record = ctx.newRecord(DATASET_CONTRIBUTOR)
+ record.setDid(did)
+ record.setName(contributor.name)
+ record.setCreator(contributor.creator)
+ record.setAffiliation(contributor.affiliation)
+ record.setEmail(contributor.email)
+ record.setComments(contributor.comments)
+ record
+ }
+ ctx.batchInsert(records.asJava).execute()
+ }
+
+ case class Contributor(
+ name: String,
+ creator: Boolean = false,
+ affiliation: String,
+ email: String,
+ comments: String
+ )
Review Comment:
`Contributor` models `affiliation`, `email`, and `comments` as non-optional `String` constructor params without defaults, but the DB schema allows them to be NULL and the tests already pass `null` for `comments`. This makes the JSON contract ambiguous and can force clients to send fields that are meant to be optional. Consider giving these fields default values (or switching to `Option[String]` in a broader refactor) so omitted fields deserialize cleanly and the type better reflects optionality.
##########
file-service/src/main/scala/org/apache/texera/service/resource/DatasetResource.scala:
##########
@@ -494,6 +564,31 @@ class DatasetResource extends LazyLogging {
}
}
+ @POST
+ @Consumes(Array(MediaType.APPLICATION_JSON))
+ @Produces(Array(MediaType.APPLICATION_JSON))
+ @RolesAllowed(Array("REGULAR", "ADMIN"))
+ @Path("/{did}/update/contributors")
+ def updateDatasetContributors(
+ @PathParam("did") did: Integer,
+ modificator: DatasetContributorsModification,
+ @Auth user: SessionUser
+ ): Response = {
Review Comment:
API path/parameter style is inconsistent with existing dataset update endpoints in this resource: `updateDatasetDescription` / `updateDatasetName` use `/update/...` with `did` in the JSON body, while `updateDatasetContributors` uses `/{did}/update/contributors` with `did` as a path param. Consider aligning this endpoint with the established pattern to reduce client-side inconsistency.
--
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(dataset): add contributor metadata storage and API [texera]
Posted by "xuang7 (via GitHub)" <gi...@apache.org>.
xuang7 commented on code in PR #6952:
URL: https://github.com/apache/texera/pull/6952#discussion_r3693808630
##########
sql/updates/29.sql:
##########
@@ -0,0 +1,38 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+\c texera_db
+
+SET search_path TO texera_db;
+
+BEGIN;
+
+CREATE TABLE IF NOT EXISTS dataset_contributor
+(
+ cid SERIAL PRIMARY KEY,
+ did INT NOT NULL,
+ name VARCHAR(256) NOT NULL,
+ creator BOOLEAN NOT NULL DEFAULT FALSE,
+ email VARCHAR(256),
+ affiliation VARCHAR(256),
+ comments TEXT,
+ FOREIGN KEY (did) REFERENCES dataset(did) ON DELETE CASCADE
Review Comment:
The solution proposed in the issue is a bit different from the current design, as the role field has been removed.
##########
file-service/src/main/scala/org/apache/texera/service/resource/DatasetResource.scala:
##########
@@ -494,6 +564,31 @@ class DatasetResource extends LazyLogging {
}
}
+ @POST
+ @Consumes(Array(MediaType.APPLICATION_JSON))
+ @Produces(Array(MediaType.APPLICATION_JSON))
+ @RolesAllowed(Array("REGULAR", "ADMIN"))
+ @Path("/{did}/update/contributors")
+ def updateDatasetContributors(
+ @PathParam("did") did: Integer,
+ modificator: DatasetContributorsModification,
+ @Auth user: SessionUser
+ ): Response = {
Review Comment:
Updated.
--
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(dataset): add contributor metadata storage and API [texera]
Posted by "xuang7 (via GitHub)" <gi...@apache.org>.
xuang7 commented on code in PR #6952:
URL: https://github.com/apache/texera/pull/6952#discussion_r3693804309
##########
file-service/src/main/scala/org/apache/texera/service/resource/DatasetResource.scala:
##########
@@ -175,12 +176,74 @@ object DatasetResource {
normalized
}
+ /**
+ * Helper function to get the contributors using the did
+ */
+ def getContributorsByDid(ctx: DSLContext, did: Integer): List[Contributor] = {
+ ctx
+ .selectFrom(DATASET_CONTRIBUTOR)
+ .where(DATASET_CONTRIBUTOR.DID.eq(did))
+ .fetch()
+ .asScala
+ .toList
+ .map { record =>
+ Contributor(
+ name = record.getName,
+ creator = record.getCreator,
+ affiliation = record.getAffiliation,
+ email = record.getEmail,
+ comments = record.getComments
+ )
+ }
+ }
+
+ /**
+ * Helper function to insert the contributors of a dataset in one batch
+ */
+ def insertContributors(ctx: DSLContext, did: Integer, contributors: List[Contributor]): Unit = {
+ if (contributors.isEmpty) {
+ return
+ }
+ val records = contributors.map { contributor =>
+ if (contributor == null || contributor.name == null || contributor.name.trim.isEmpty) {
+ throw new BadRequestException("Each contributor must have a name")
+ }
+ if (
+ contributor.name.length > 256 ||
+ Option(contributor.email).exists(_.length > 256) ||
+ Option(contributor.affiliation).exists(_.length > 256)
+ ) {
+ throw new BadRequestException("Contributor fields must not exceed 256 characters")
+ }
+ val record = ctx.newRecord(DATASET_CONTRIBUTOR)
+ record.setDid(did)
+ record.setName(contributor.name)
+ record.setCreator(contributor.creator)
+ record.setAffiliation(contributor.affiliation)
+ record.setEmail(contributor.email)
+ record.setComments(contributor.comments)
+ record
+ }
+ ctx.batchInsert(records.asJava).execute()
+ }
+
+ case class Contributor(
+ name: String,
+ creator: Boolean = false,
+ affiliation: String,
+ email: String,
+ comments: String
+ )
Review Comment:
Updated.
--
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(dataset): add contributor metadata storage and API [texera]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #6952:
URL: https://github.com/apache/texera/pull/6952#issuecomment-5097926046
<!-- texera-benchmarks-comment -->
## ⚠️ Benchmark changes need a look
🟢 2 better · 🔴 5 worse · ⚪ 8 noise (<±5%) · 0 without baseline
> Compared against main `31f1e7b` 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/30313462038)
| | config | throughput | MB/s | latency | max Δ latest / 7d |
|---|---|---:|---:|---:|---:|
| 🔴 | bs=10 sw=10 sl=64 | 383 | 0.234 | 25,567/33,728/33,728 us | 🔴 +21.5% / 🔴 +107.8% |
| 🟢 | bs=100 sw=10 sl=64 | 833 | 0.508 | 118,235/133,709/133,709 us | 🟢 -11.5% / 🔴 +23.1% |
| ⚪ | bs=1000 sw=10 sl=64 | 953 | 0.582 | 1,050,028/1,088,475/1,088,475 us | ⚪ within ±5% / 🔴 -5.1% |
<details><summary>Baseline details</summary>
Latest main `31f1e7b` from same runner
| config | metric | PR | latest main | 7d avg | Δ latest | Δ 7d |
|---|---|---:|---:|---:|---:|---:|
| bs=10 sw=10 sl=64 | throughput | 383 tuples/sec | 457 tuples/sec | 767.9 tuples/sec | -16.2% | -50.1% |
| bs=10 sw=10 sl=64 | MB/s | 0.234 MB/s | 0.279 MB/s | 0.469 MB/s | -16.1% | -50.1% |
| bs=10 sw=10 sl=64 | p50 | 25,567 us | 21,330 us | 12,502 us | +19.9% | +104.5% |
| bs=10 sw=10 sl=64 | p95 | 33,728 us | 27,764 us | 16,234 us | +21.5% | +107.8% |
| bs=10 sw=10 sl=64 | p99 | 33,728 us | 27,764 us | 18,919 us | +21.5% | +78.3% |
| bs=100 sw=10 sl=64 | throughput | 833 tuples/sec | 836 tuples/sec | 974.8 tuples/sec | -0.4% | -14.5% |
| bs=100 sw=10 sl=64 | MB/s | 0.508 MB/s | 0.51 MB/s | 0.595 MB/s | -0.4% | -14.6% |
| bs=100 sw=10 sl=64 | p50 | 118,235 us | 116,762 us | 102,449 us | +1.3% | +15.4% |
| bs=100 sw=10 sl=64 | p95 | 133,709 us | 151,101 us | 108,652 us | -11.5% | +23.1% |
| bs=100 sw=10 sl=64 | p99 | 133,709 us | 151,101 us | 116,310 us | -11.5% | +15.0% |
| bs=1000 sw=10 sl=64 | throughput | 953 tuples/sec | 956 tuples/sec | 1,004 tuples/sec | -0.3% | -5.1% |
| bs=1000 sw=10 sl=64 | MB/s | 0.582 MB/s | 0.583 MB/s | 0.613 MB/s | -0.2% | -5.0% |
| bs=1000 sw=10 sl=64 | p50 | 1,050,028 us | 1,044,502 us | 999,606 us | +0.5% | +5.0% |
| bs=1000 sw=10 sl=64 | p95 | 1,088,475 us | 1,095,818 us | 1,046,770 us | -0.7% | +4.0% |
| bs=1000 sw=10 sl=64 | p99 | 1,088,475 us | 1,095,818 us | 1,076,937 us | -0.7% | +1.1% |
</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,522.35,200,128000,383,0.234,25566.56,33727.71,33727.71
1,100,10,64,20,2402.14,2000,1280000,833,0.508,118234.85,133709.45,133709.45
2,1000,10,64,20,20979.01,20000,12800000,953,0.582,1050028.12,1088474.63,1088474.63
```
</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] feat(dataset): add contributor metadata storage and API [texera]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #6952:
URL: https://github.com/apache/texera/pull/6952#issuecomment-5097912718
## [Codecov](https://app.codecov.io/gh/apache/texera/pull/6952?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.93%. Comparing base ([`8c7089b`](https://app.codecov.io/gh/apache/texera/commit/8c7089b3fa247087c40817ca55c917505313c9f3?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`e3c36cb`](https://app.codecov.io/gh/apache/texera/commit/e3c36cbbfc1185bdca026a984eb3d012857c2d52?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
:warning: Report is 2 commits behind head on main.
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## main #6952 +/- ##
============================================
- Coverage 78.93% 78.93% -0.01%
Complexity 3784 3784
============================================
Files 1160 1160
Lines 46049 46047 -2
Branches 5110 5110
============================================
- Hits 36350 36348 -2
Misses 8078 8078
Partials 1621 1621
```
| [Flag](https://app.codecov.io/gh/apache/texera/pull/6952/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/6952/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/6952/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 [8c7089b](https://app.codecov.io/gh/apache/texera/commit/8c7089b3fa247087c40817ca55c917505313c9f3?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/6952/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `72.17% <ø> (ø)` | | Carriedforward from [8c7089b](https://app.codecov.io/gh/apache/texera/commit/8c7089b3fa247087c40817ca55c917505313c9f3?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/6952/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `20.49% <ø> (ø)` | | |
| [config-service](https://app.codecov.io/gh/apache/texera/pull/6952/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/6952/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `67.07% <ø> (-0.14%)` | :arrow_down: | Carriedforward from [8c7089b](https://app.codecov.io/gh/apache/texera/commit/8c7089b3fa247087c40817ca55c917505313c9f3?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/6952/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `82.91% <ø> (ø)` | | Carriedforward from [8c7089b](https://app.codecov.io/gh/apache/texera/commit/8c7089b3fa247087c40817ca55c917505313c9f3?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/6952/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `78.94% <ø> (ø)` | | |
| [pyamber](https://app.codecov.io/gh/apache/texera/pull/6952/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 [8c7089b](https://app.codecov.io/gh/apache/texera/commit/8c7089b3fa247087c40817ca55c917505313c9f3?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/6952/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `26.31% <ø> (ø)` | | |
*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/6952?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] feat(dataset): add contributor metadata storage and API [texera]
Posted by "xuang7 (via GitHub)" <gi...@apache.org>.
xuang7 merged PR #6952:
URL: https://github.com/apache/texera/pull/6952
--
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(dataset): add contributor metadata storage and API [texera]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #6952:
URL: https://github.com/apache/texera/pull/6952#issuecomment-5097885843
<!-- 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:** `@kunwp1`, `@Neilk1021`
You can notify them by mentioning `@kunwp1`, `@Neilk1021` 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