Skip to content

Conversation

steveloughran
Copy link
Contributor

@steveloughran steveloughran commented Jun 19, 2025

Compilation failure during change to source class between #7379 PR and commit.

Not running Itests on this as its an emergency fix, test does pass, so merging immediately

[INFO] -------------------------------------------------------
[INFO] Running org.apache.hadoop.fs.s3a.impl.TestAwsClientConfig
[INFO] Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.232 s - in org.apache.hadoop.fs.s3a.impl.TestAwsClientConfig
[INFO] 
[INFO] Results:

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
  • Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
  • If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

Compilation failure during change to source class between PR and commit.
@steveloughran steveloughran merged commit ad7aa0e into apache:trunk Jun 19, 2025
1 of 2 checks passed
@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 0s Docker mode activated.
-1 ❌ patch 0m 21s #7751 does not apply to trunk. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.
Subsystem Report/Notes
GITHUB PR #7751
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7751/1/console
versions git=2.34.1
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

YanivKunda pushed a commit to YanivKunda/hadoop that referenced this pull request Jul 7, 2025
Followup to apache#7379

Compilation failure due to import change to in test source between PR and commit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants