tailcallhq / tailcall

High Performance GraphQL Runtime
https://tailcall.run
Apache License 2.0
1.27k stars 242 forks source link

Support for GRPC #414

Closed tusharmath closed 9 months ago

tusharmath commented 12 months ago

Tailcall currently supports HTTP as it's only data-source. Being a generic GraphQL proxy, it should be able to load data over GRPC also.

The user experience should match that of @http as much as possible.

schema @grpc(host: "grpc.service.com", port: 8080) {
  query: Query
  mutation: Mutation
}

type Query {
  user(id: ID!): User @grpc(
    service: "UserService", 
    method: "GetUser", 
    request: {user_id: "{{args.id}}"}
  )
}

Technical Requirements

Additional Notes

algora-pbc[bot] commented 12 months ago

💎 $350 bounty created by tailcallhq 🙋 If you'd like to work on this issue, comment below to get assigned 👉 To claim this bounty, submit a pull request that includes the text /claim #414 somewhere in its body 📝 Before proceeding, please make sure you can receive payouts in your country 💵 Payment arrives in your account 2-5 days after the bounty is rewarded 💯 You keep 100% of the bounty award 🙏 Thank you for contributing to tailcallhq/tailcall! 🙋‍♂️ Join our discord channel if you need help.

ologbonowiwi commented 12 months ago

I found hyperium/tonic; likely, we can use it.

deepakdinesh1123 commented 11 months ago

@tusharmath Is this issue still available for work?

tusharmath commented 11 months ago

@deepakdinesh1123 Yes, all tickets are available until closed ;)

deepakdinesh1123 commented 11 months ago

@tusharmath I have added a specification for GRPC support in #530 please take a look at it

thaodt commented 11 months ago

/attempt #414

Options
algora-pbc[bot] commented 11 months ago

The bounty is up for grabs! Everyone is welcome to /attempt #414 🙌

thaodt commented 10 months ago

because I'm engaging with another issue, so no time for working on this one. Anyone feel free to pick it, I will be here to help on guidance and code review if you want.

ssddOnTop commented 10 months ago

@tusharmath I think I am done with #441 (review pending) Can you assign this one to me

/attempt #414

algora-pbc[bot] commented 10 months ago

@ssddOnTop: The Tailcall Inc. team prefers to assign a single contributor to the issue rather than let anyone attempt it right away. We recommend waiting for a confirmation from a member before getting started.

A-N-uraag commented 10 months ago

@tusharmath throwing my 🎩 in since there was no activity since 4 days. Can I get assigned?

tusharmath commented 10 months ago

@ssddOnTop Are you working on this?

ssddOnTop commented 10 months ago

@ssddOnTop Are you working on this?

Yes

ssddOnTop commented 10 months ago

Still stuck on a few things, I'll make a PR in a few days

ssddOnTop commented 10 months ago

@tusharmath sorry I won't be able to do grpc integration

AayushMohan commented 10 months ago

Hey @tusharmath, can you please assign this to me?

tusharmath commented 10 months ago

This issue has been attempted multiple times before and never reached completion. I would recommend only folk who have a good understanding of the project and grpc in general to attempt it. @A-N-uraag @AayushMohan let me know if you feel confident about implementing this feature. Alternatively working as a team with split bounty could also work.

Rutik7066 commented 9 months ago

Hi @tusharmath i can. And would like to work on this issue.

amitksingh1490 commented 9 months ago

@Rutik7066 Please Go ahead assigning this to you

Rutik7066 commented 9 months ago

Thank you @amitksingh1490

tusharmath commented 9 months ago

We are removing bounty from this PR. It's going to be taken up internally for now. Thank you everyone for trying 🙏

Rutik7066 commented 9 months ago

No problem @tusharmath. I have researched and started learning package call tonic for grpc. And was looking to complete the work by this weekend.

Rutik7066 commented 9 months ago

Here is the doc https://docs.rs/tonic/latest/tonic/ it may help you.

tusharmath commented 9 months ago

Feel free to connect with @amitksingh1490 who has made a lot of progress in his PR.

tusharmath commented 9 months ago

fixed in #730