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

Add getStatus RESTful API #18312

Merged
merged 1 commit into from
Oct 25, 2023
Merged

Conversation

JiamingMai
Copy link
Contributor

@JiamingMai JiamingMai commented Oct 21, 2023

Add getStatus RESTful API.

Example:
Get the specified directory/file information by the following request:
curl -X GET http://localhost:28080/v1/info?path=/tpcds-data

The response JSON looks like:
[ { "mType": "directory", "mName": "tpcds-data", "mPath": "/tpcds-data", "mUfsPath": "s3a://jiamingmai-test/tpcds-data", "mLastModificationTimeMs": 0, "mLength": 0, "mHumanReadableFileSize": "0B" } ]

image

@JiamingMai JiamingMai self-assigned this Oct 21, 2023
@JiamingMai JiamingMai added the type-feature This issue is a feature request label Oct 21, 2023
@LuQQiu
Copy link
Contributor

LuQQiu commented Oct 23, 2023

@JiamingMai Thanks for adding the API call! Could you help support the full ufs path similar to listdir files RESTful API?
From Python perspective, we always provide the full ufs path as path parameter, thanks!

@JiamingMai
Copy link
Contributor Author

@JiamingMai Thanks for adding the API call! Could you help support the full ufs path similar to listdir files RESTful API? From Python perspective, we always provide the full ufs path as path parameter, thanks!

This API also supports full UFS path as path parameter.

Copy link
Contributor

@LuQQiu LuQQiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for supporting the feature

@JiamingMai
Copy link
Contributor Author

alluxio-bot, merge this please.

@alluxio-bot alluxio-bot merged commit 5b0e661 into Alluxio:main Oct 25, 2023
14 checks passed
@JiamingMai JiamingMai deleted the add-info-restful-api branch November 20, 2023 10:34
ssz1997 pushed a commit to ssz1997/alluxio that referenced this pull request Dec 15, 2023
Add getStatus RESTful API.

**Example:**
Get the specified directory/file information by the following request:
`curl -X GET http://localhost:28080/v1/info?path=/tpcds-data`

The response JSON looks like:
`[
  {
    "mType": "directory",
    "mName": "tpcds-data",
    "mPath": "/tpcds-data",
    "mUfsPath": "s3a://jiamingmai-test/tpcds-data",
    "mLastModificationTimeMs": 0,
    "mLength": 0,
    "mHumanReadableFileSize": "0B"
  }
]`

<img width="597" alt="image" src="https://github.com/Alluxio/alluxio/assets/6129818/b38d644b-11da-4206-937b-6d61fd6b3a6c">

			pr-link: Alluxio#18312
			change-id: cid-6e563a21372e9fe1867a36d9b311e246b49c459e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-feature This issue is a feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants