Minor refactoring in Redis connector#11957
Conversation
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla. |
plugin/trino-redis/src/main/java/io/trino/plugin/redis/RedisDataType.java
Outdated
Show resolved
Hide resolved
plugin/trino-redis/src/test/java/io/trino/plugin/redis/TestRedisConnectorTest.java
Outdated
Show resolved
Hide resolved
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla. |
|
@ebyhr Hi ebyhr, I submitted a cla yesterday, but I haven't received a reply yet, so the current pr submission may fail the verification, which can be ignored for now. Thank you very much! |
plugin/trino-redis/src/main/java/io/trino/plugin/redis/RedisDataType.java
Outdated
Show resolved
Hide resolved
ebyhr
added
the
no-release-notes
This also removed the `SET` data type in RedisDataType because it is not used.
polaris6
force-pushed
the
adjust-redis-connector
branch
from
e869d55 to
331fea2
Compare
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla. |
|
@ebyhr Thank you for your suggestion, I resubmitted it. |
|
@polaris6 Contributors list was updated today, but your account doesn't exist. Just in case, did you send CLA with |
|
Hi @ebyhr, the account name is correct, but there is a problem with the signature, I have re-sent the email to Martin, thank you very much for the reminder. |
|
@cla-bot check |
|
The cla-bot has been summoned, and re-checked this pull request! |
|
Thanks! |
3 participants
No description provided.