Skip to content

HBASE-30082 Upgrade hbase-server to use junit5 Part12 (#8190)#8253

Open
Apache9 wants to merge 3 commits into
apache:branch-2from
Apache9:HBASE-30082-branch-2
Open

HBASE-30082 Upgrade hbase-server to use junit5 Part12 (#8190)#8253
Apache9 wants to merge 3 commits into
apache:branch-2from
Apache9:HBASE-30082-branch-2

Conversation

@Apache9
Copy link
Copy Markdown
Contributor

@Apache9 Apache9 commented May 18, 2026

(cherry picked from commit 61e78cd)

Signed-off-by: Xiao Liu <liuxiaocs@apache.org>
(cherry picked from commit 61e78cd)
@Apache9 Apache9 added the backport This PR is a back port of some issue or issues already committed to master label May 18, 2026
@Apache9
Copy link
Copy Markdown
Contributor Author

Apache9 commented May 19, 2026

@ndimiduk Could you please take a look at this? I'm not sure what is the problem here...

  Processing: GH:8253
  GITHUB PR #8253 is being downloaded from
  https://api.github.com/repos/apache/hbase/pulls/8253
    JSON data at Mon May 18 16:16:13 UTC 2026
    Patch data at Mon May 18 16:16:14 UTC 2026
  [Mon May 18 16:16:15 UTC 2026 DEBUG]: github_locate_patch: not a github pull request.
  [Mon May 18 16:16:15 UTC 2026 DEBUG]: gitlab: GH:8253 is not a merge request #
  [Mon May 18 16:16:15 UTC 2026 DEBUG]: generic_locate_patch: failed to download the patch.
  ERROR: Unsure how to process GH:8253. Permissions missing?

@ndimiduk
Copy link
Copy Markdown
Member

HTTP 406 Not Acceptable. GitHub's API refuses to generate the patch for this PR. Let me confirm the response body.

Let me try again with Yetus HEAD.

@ndimiduk ndimiduk added the test-yetus-prerelease Test the precommit run against an unreleased apache/yetus revision. See HBASE-30133. label May 19, 2026
@ndimiduk
Copy link
Copy Markdown
Member

{
  "message": "Sorry, the diff exceeded the maximum number of lines (20000)",
  "errors": [
    {
      "resource": "PullRequest",
      "field": "diff",
      "code": "too_large"
    }
  ],
  "documentation_url": "https://docs.github.com/rest/pulls/pulls#get-a-pull-request",
  "status": "406"
}

@Apache9
Copy link
Copy Markdown
Contributor Author

Apache9 commented May 19, 2026

So the only way is to split the PR to several smaller ones?

@ndimiduk
Copy link
Copy Markdown
Member

So the only way is to split the PR to several smaller ones?

In Yetus 0.15, yes. But I fixed this in YETUS-1263, so I expect that building against Yetus HEAD should process this PR correctly, we'll see in a few hours.

@Apache9
Copy link
Copy Markdown
Contributor Author

Apache9 commented May 19, 2026

Seems the script for generating job summary is broken on the latest yetus...

@ndimiduk
Copy link
Copy Markdown
Member

Seems the script for generating job summary is broken on the latest yetus...

Sorry, but what are you looking at specifically? https://github.com/apache/hbase/actions/runs/26086185939?pr=8253 looks fine to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a back port of some issue or issues already committed to master test-yetus-prerelease Test the precommit run against an unreleased apache/yetus revision. See HBASE-30133.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants