Skip to content

Conversation

@kou
Copy link
Member

@kou kou commented Apr 10, 2025

Rationale for this change

uploaded-files.txt is for Artifactory. Artifactory doesn't have the staging feature. So we emulate the feature by uploaded-files.txt.

But Maven repository has the feature. So we don't need to generate uploaded-files.txt.

What changes are included in this PR?

Don't generate uploaded-files.txt.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

@github-actions
Copy link

⚠️ GitHub issue #46081 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Apr 10, 2025
@kou
Copy link
Member Author

kou commented Apr 11, 2025

+1

@kou kou merged commit fd1919f into apache:main Apr 11, 2025
5 checks passed
@kou kou removed the awaiting committer review Awaiting committer review label Apr 11, 2025
@kou kou deleted the release-binary-upload-needless-files branch April 11, 2025 05:45
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit fd1919f.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 4 possible false positives for unstable benchmarks that are known to sometimes produce them.

assignUser pushed a commit that referenced this pull request Apr 15, 2025
…Maven repository (#46082)

### Rationale for this change

`uploaded-files.txt` is for Artifactory. Artifactory doesn't have the staging feature. So we emulate the feature by `uploaded-files.txt`.

But Maven repository has the feature. So we don't need to generate `uploaded-files.txt`.

### What changes are included in this PR?

Don't generate `uploaded-files.txt`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: #46081

Authored-by: Sutou Kouhei <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
zanmato1984 pushed a commit to zanmato1984/arrow that referenced this pull request Apr 15, 2025
…` for Maven repository (apache#46082)

### Rationale for this change

`uploaded-files.txt` is for Artifactory. Artifactory doesn't have the staging feature. So we emulate the feature by `uploaded-files.txt`.

But Maven repository has the feature. So we don't need to generate `uploaded-files.txt`.

### What changes are included in this PR?

Don't generate `uploaded-files.txt`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: apache#46081

Authored-by: Sutou Kouhei <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant