feast-dev / feast-java-old

Feast Java Components
Apache License 2.0
12 stars 29 forks source link

Add auth support for Redis #43

Closed xiaoyongzhumsft closed 3 years ago

xiaoyongzhumsft commented 3 years ago

This PR adds password support when reading from Redis. More specifically, it utilizes the password field in lettuce, and users can specify a field called password in yaml file to authenticate for Redis.

achals commented 3 years ago

Thanks for the contribution @snowmanmsft ! I think you need to fix DCO, but beyond that the change looks good to me.

xiaoyongzhumsft commented 3 years ago

Thanks for the contribution @snowmanmsft ! I think you need to fix DCO, but beyond that the change looks good to me.

Thanks! I've fixed it.