Skip to content

Replace /id api call with /namespace#89

Merged
hailihu merged 2 commits intomasterfrom
replace-id-api-call
Jun 16, 2025
Merged

Replace /id api call with /namespace#89
hailihu merged 2 commits intomasterfrom
replace-id-api-call

Conversation

@onnozweers
Copy link
Contributor

With ada --checksum and ada --stat, the API's /id call is used; mainly for historical reasons. But the /id API call will soon be changed. Anyway, we don't need it anymore, because we can now get the same info with the /namespace call, which even saves one API operation per file.

This change replaces Ada's /id calls with /namespace calls. The output of ada --checksum is still the same. The output of ada --stat will probably be a bit different.

onnozweers added 2 commits May 2, 2025 17:28
* It's simpler
* It's faster (it requires only one API call instead of two
* The dCache devs will soon make the /id call admin-only
* It's simpler
* It's faster (it requires only one API call instead of two
* The dCache devs will soon make the /id call admin-only
@onnozweers onnozweers requested a review from hailihu May 2, 2025 15:47
@hailihu
Copy link
Contributor

hailihu commented Jun 16, 2025

Indeed, the output of ada --checksum is the same, while there is a minor difference in output of ada --stat. This seems acceptable as the information is almost identical.

@hailihu hailihu merged commit b866b20 into master Jun 16, 2025
1 check passed
@hailihu hailihu deleted the replace-id-api-call branch June 17, 2025 07:30
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.

2 participants