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

[Improvement] Add how to develop Gravitino trino connector for new developers #1193

Closed
yuqi1129 opened this issue Dec 18, 2023 · 3 comments · Fixed by #2446
Closed

[Improvement] Add how to develop Gravitino trino connector for new developers #1193

yuqi1129 opened this issue Dec 18, 2023 · 3 comments · Fixed by #2446
Labels
improvement Improvements on everything

Comments

@yuqi1129
Copy link
Contributor

What would you like to be improved?

There seem to be no instructions on how to debug and develop the Gravitino Trino connector locally for new developers. We need to add some documents about it.

How should we improve?

No response
@yuqi1129 yuqi1129 added the improvement Improvements on everything label Dec 18, 2023
@xiacongling
Copy link
Contributor

Hi, @yuqi1129. I had to compile the connector and then build a docker image for debugging. Is there any easier way to quickly start a development server? Can you share your experience about that?

@yuqi1129
Copy link
Contributor Author

yuqi1129 commented Mar 5, 2024

@xiacongling
Well, I'll write a doc on how to debug and create "Trino-connector" locally. Please take a look at this.

This was referenced Mar 6, 2024
Closed
Merged
@yuqi1129
Copy link
Contributor Author

yuqi1129 commented Mar 7, 2024

@xiacongling
The related PR is here #2446, please take a look, thanks.

yuqi1129 added a commit that referenced this issue Mar 14, 2024
@yuqi1129
[#1193] improvement(docs): Add the document about how to debug trino …
b154519 
…connector locally. (#2446)

### What changes were proposed in this pull request?

Add a document about how to debug develop and debug `trino-connector`
locally.

### Why are the changes needed?

To make users more knowledgeable about Gravitino trino connector.

Fix: #1193 

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

N/A.
yijhenlin pushed a commit to yijhenlin/gravitino that referenced this issue Mar 14, 2024
@yuqi1129 @yijhenlin
[apache#1193] improvement(docs): Add the document about how to debug …
0c5be7e 
…trino connector locally. (apache#2446)

### What changes were proposed in this pull request?

Add a document about how to debug develop and debug `trino-connector`
locally.

### Why are the changes needed?

To make users more knowledgeable about Gravitino trino connector.

Fix: apache#1193 

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

N/A.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
improvement Improvements on everything
Projects
None yet
Milestone
No milestone
2 participants
@xiacongling @yuqi1129