Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace AlluxioURI with UfsUrl #17863

Closed

Conversation

YichuanSun
Copy link
Contributor

What changes are proposed in this pull request?

First step for replacing AlluxioURI with UfsUrl. This PR is focus on UfsUrl class design and implementation.

Why are the changes needed?

Please clarify why the changes are needed. For instance,

  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?

Please list the user-facing changes introduced by your change, including

  1. change in user-facing APIs
  2. addition or removal of property keys
  3. webui

jiacheliu3 and others added 15 commits June 14, 2023 14:47
### What changes are proposed in this pull request?
Disable keep-alive for HTTP requests to deal with connection resets when fetching maven deps from central repo. Unable to repro ([ref](https://stackoverflow.com/questions/55899091/maven-retry-dependency-download-if-failed))

### Why are the changes needed?
Trying to address tarball build failures via CI due to maven errors. 

### Does this PR introduce any user facing changes?
no

			pr-link: Alluxio#17864
			change-id: cid-767a7869d93ca2dbaee4e036d2bd7779ab7618e9
### What changes are proposed in this pull request?

Please outline the changes and how this PR fixes the issue.

### Why are the changes needed?

Please clarify why the changes are needed. For instance,
  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, describe the bug.

### Does this PR introduce any user facing changes?

Please list the user-facing changes introduced by your change, including
  1. change in user-facing APIs
  2. addition or removal of property keys
  3. webui

			pr-link: Alluxio#17866
			change-id: cid-b6c2036ad76c65854135d660a2de1ca486455298
### What changes are proposed in this pull request?

Please outline the changes and how this PR fixes the issue.

### Why are the changes needed?

Please clarify why the changes are needed. For instance,
  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, describe the bug.

### Does this PR introduce any user facing changes?

Please list the user-facing changes introduced by your change, including
  1. change in user-facing APIs
  2. addition or removal of property keys
  3. webui

			pr-link: Alluxio#17866
			change-id: cid-b6c2036ad76c65854135d660a2de1ca486455298
### What changes are proposed in this pull request?

Fix load cli doc and remove job service doc

### Why are the changes needed?

doc quality
### Does this PR introduce any user facing changes?
na

			pr-link: Alluxio#17868
			change-id: cid-628e4b5b68919245a7bcdbd3ae859c0bdc6a2e5f
YichuanSun and others added 4 commits August 1, 2023 12:58
…solved todo. Delete UfsUrlUtils.java for it is not be used.
### What changes are proposed in this pull request?

Add multipart upload interface & 
add HDFS unit tests using HDFS minicluster


### Why are the changes needed?

To increase code quality

### Does this PR introduce any user facing changes?

Please list the user-facing changes introduced by your change, including
  1. change in user-facing APIs
  2. addition or removal of property keys
  3. webui

			pr-link: Alluxio#17862
			change-id: cid-e01bdb47bdeb9218354040f69587a3550972bdc5
@alluxio-bot
Copy link
Contributor

Automated checks report:

  • PR title follows the conventions: FAIL
    • The title of the PR does not pass all the checks. Please fix the following issues:
      • Title must not end with punctuation
  • Commits associated with Github account: PASS

Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks.

@YichuanSun YichuanSun changed the title Replace AlluxioURI with UfsUrl. Replace AlluxioURI with UfsUrl Aug 2, 2023
@alluxio-bot
Copy link
Contributor

Automated checks report:

  • PR title follows the conventions: PASS
  • Commits associated with Github account: PASS

All checks passed!

@YichuanSun YichuanSun closed this Aug 17, 2023
@YichuanSun YichuanSun deleted the ufs-url-yichuan-repair branch October 9, 2023 03:11
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.

7 participants