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

Import code from lyft/presto-gateway #4

Merged
merged 1 commit into from
Aug 16, 2023
Merged

Conversation

vishalya
Copy link
Member

This is the base version for trino-gateway, it is a copy of the code from https://github.com/lyft/presto-gateway v1.9.5 (commit 69ac14b). This code is available under Apache License 2.0 that allows me to copy this version and submit it as a PR for the trino-gateway project run by trino community.

I along with the Trino community acknowledge the contributions from the authors of this code and thank them for their time and effort which helped us start with trino-gateway.
The full list of contributors is here - https://github.com/lyft/presto-gateway/graphs/contributors

@cla-bot cla-bot bot added the cla-signed label Aug 15, 2023
@mosabua
Copy link
Member

mosabua commented Aug 15, 2023

Could you actually reword the commit message to the title

"Import code from lyft/presto-gateway"

and have all the details from the PR description as bullet item points in the commit message? That way the reference is in the git repo history rather than just the PR..

- This is the base version for trino-gateway, it is a copy of the code from [https://github.com/lyft/presto-gateway v1.9.5](https://github.com/lyft/presto-gateway/tree/69ac14b888c9caff96daa7d22d2354adbc2a87de) (commit 69ac14b).
- This code is available under [Apache License 2.0](https://github.com/lyft/presto-gateway/blob/69ac14b888c9caff96daa7d22d2354adbc2a87de/LICENSE) that allows me to copy this version and submit it as a PR for the trino-gateway project run by trino community.

- I along with the Trino community acknowledge the contributions from the authors of this code and thank them for their time and effort which helped us start with trino-gateway.
- The full list of contributors is here - https://github.com/lyft/presto-gateway/graphs/contributors
@vishalya vishalya changed the title Base commit for trino-gateway Import code from lyft/presto-gateway Aug 16, 2023
@vishalya
Copy link
Member Author

Changed the commit message as well as the PR title.

@mosabua mosabua merged commit 51f06c8 into trinodb:main Aug 16, 2023
1 check passed
@mosabua
Copy link
Member

mosabua commented Aug 16, 2023

Merging as is .. code improvements and more will follow later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants