kyuubi icon indicating copy to clipboard operation
kyuubi copied to clipboard

SparkConnectEngineDemo

Open davidyuan1223 opened this issue 1 year ago โ€ข 2 comments

:mag: Description

Issue References ๐Ÿ”—

This pull request fixes #

Describe Your Solution ๐Ÿ”ง

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Types of changes :bookmark:

  • [ ] Bugfix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan ๐Ÿงช

Behavior Without This Pull Request :coffin:

Behavior With This Pull Request :tada:

Related Unit Tests


Checklist ๐Ÿ“

Be nice. Be informative.

davidyuan1223 avatar May 06 '24 17:05 davidyuan1223

@pan3793 @yaooqinn @ulysses-you this pr is only a draft for discussion. What do you think about this method to compatible spark grpc ?

davidyuan1223 avatar May 06 '24 17:05 davidyuan1223

Thank you @davidyuan1223 . We should add the grpc frontend service first, it is used to receive spark connect client request and send to connect engine.

ulysses-you avatar May 07 '24 03:05 ulysses-you

@ulysses-you what do you think about the new module kyuubi-grpc(only structure, not write test case yet)

davidyuan1223 avatar May 11 '24 02:05 davidyuan1223

@ulysses-you what do you think about the new module kyuubi-grpc(only structure, not write test case yet)

@davidyuan1223 yeah, a new module for kyuubi grpc server is better. We can refactor and improve code after completing the connect frontend in future.

ulysses-you avatar May 11 '24 06:05 ulysses-you