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.

Sign up for GitHub

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

Jump to bottom

K2: add TL types and functions for HTTP queries #1121

Merged
merged 13 commits into from
Oct 22, 2024
Merged

K2: add TL types and functions for HTTP queries #1121

merged 13 commits into from
Oct 22, 2024

Conversation

apolyakov
Copy link
Contributor

This PR also does some small refactoring of K2 TL.

@apolyakov apolyakov added runtime Feature related to runtime k2 k2 related labels Oct 17, 2024
@apolyakov apolyakov added this to the next milestone Oct 17, 2024
@apolyakov apolyakov self-assigned this Oct 17, 2024
@DrDet DrDet removed this from the next milestone Oct 17, 2024
mkornaukhov03
mkornaukhov03 requested changes Oct 22, 2024
View reviewed changes
Copy link
Contributor

@mkornaukhov03 mkornaukhov03 left a comment

Choose a reason for hiding this comment

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

I've seen only tl-stuff, haven't checked semantics of http things

runtime-light/tl/tl-types.h Show resolved Hide resolved
runtime-light/tl/tl-types.h Show resolved Hide resolved
runtime-light/stdlib/rpc/rpc-api.cpp Outdated Show resolved Hide resolved
runtime-light/stdlib/job-worker/job-worker-api.cpp Show resolved Hide resolved
mkornaukhov03
mkornaukhov03 approved these changes Oct 22, 2024
View reviewed changes
Copy link
Contributor

@mkornaukhov03 mkornaukhov03 left a comment

Choose a reason for hiding this comment

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

ok

@apolyakov apolyakov merged commit c5b72aa into master Oct 22, 2024
5 checks passed
@apolyakov apolyakov deleted the apolyakov/k2-http-tl branch October 22, 2024 14:37
@Danil42Russia Danil42Russia added this to the next milestone Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mkornaukhov03 mkornaukhov03 mkornaukhov03 approved these changes

@DrDet DrDet Awaiting requested review from DrDet

@astrophysik astrophysik Awaiting requested review from astrophysik

Assignees

@apolyakov apolyakov

Labels
k2 k2 related runtime Feature related to runtime
Projects
None yet
Milestone
next
Development

Successfully merging this pull request may close these issues.
4 participants
@apolyakov @mkornaukhov03 @Danil42Russia @DrDet