Skip to content

Comments

Optimize Get-Contents API#2873

Open
takezoe wants to merge 1 commit intomasterfrom
optimize-contents-api-2
Open

Optimize Get-Contents API#2873
takezoe wants to merge 1 commit intomasterfrom
optimize-contents-api-2

Conversation

@takezoe
Copy link
Member

@takezoe takezoe commented Sep 16, 2021 •
edited
Loading

A successor of #2686 which fixes #2674 and #2858

JGitUtils.getFileList() is very slow depending on the number of files. This pull request avoids it as much as possible.

Before submitting a pull-request to GitBucket I have first:

  • read the contribution guidelines
  • rebased my branch over master
  • verified that project is compiling
  • verified that tests are passing
  • squashed my commits as appropriate (keep several commits if it is relevant to understand the PR)
  • marked as closed using commit message all issue ID that this PR should correct

@takezoe takezoe mentioned this pull request Sep 16, 2021
Closed
6 tasks
@takezoe takezoe added APIv3 GitHub compatible API improvement labels Sep 16, 2021
@takezoe takezoe force-pushed the optimize-contents-api-2 branch from 7fed834 to c224e12 Compare September 16, 2021 17:45
@takezoe takezoe force-pushed the optimize-contents-api-2 branch from c224e12 to fab6026 Compare November 5, 2021 10:39
@takezoe takezoe force-pushed the optimize-contents-api-2 branch from fab6026 to 9cc3a0d Compare November 5, 2021 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

APIv3 GitHub compatible API improvement

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

API request to contents/ is slow

1 participant

@takezoe