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

[#465] introduce irods.experimental.client.http module #466

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

d-w-moore
Copy link
Collaborator

irods.client.experimental.http.Session is a new way of connecting and reflects one possible future of iRODS clients, ie communicating via HTTP instead of the traditional iRODS protocol.

irods.client.http.Session is a new way of connecting and reflects one
possible future of iRODS clients, ie communicating via HTTP instead of
the traditional iRODS protocol.
@d-w-moore d-w-moore marked this pull request as draft July 19, 2023 17:42
@d-w-moore d-w-moore changed the title [#465] introduce irods.client..experimental.http module [#465] introduce irods.client.experimental.http module Jul 19, 2023
@trel
Copy link
Member

trel commented Jul 19, 2023

experimental, .... 'above' client ?

@trel
Copy link
Member

trel commented Jul 19, 2023

and commit message to be updated/corrected once decided.

@korydraughn
Copy link
Contributor

I vote for experimental being above client as well.

That gives us a single namespace for any number of experimental things.

We also need to include documentation that explains the purpose/goals of the experimental module.

@d-w-moore
Copy link
Collaborator Author

Yes I suppose that move makes sense.

(See Session.data_object_replicas).

Also, swap the client and experimental namespaces.
And rework the demo slightly.
@d-w-moore d-w-moore changed the title [#465] introduce irods.client.experimental.http module [#465] introduce irods.experimental.client.http module Jul 19, 2023
regularize use of int and string for offset, count

data_object gets one replica by default

can pass genquery options to session.data_object
offset no longer used to indicate return type.
len of page properly used to update offset in each call to get_r
to get initial page, we can call next_page() init on returned iterator.
other offsets of pages should be done Pythonically ie with itertools.islice
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants