[python] How to define redis operator ttl

[jairhenrique]

When I try to define a default_ttl on redis operator I get an error.

import opendal
op = opendal.Operator('redis', host='localhost:6379', default_ttl=30)

thread '<unnamed>' panicked at src/operator.rs:64:22:
must be valid hashmap: PyErr { type: <class 'TypeError'>, value: TypeError("'int' object cannot be converted to 'PyString'"), traceback: None }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
---------------------------------------------------------------------------
PanicException                            Traceback (most recent call last)
Cell In[4], line 1
----> 1 op = opendal.Operator('redis', host='localhost:6379', default_ttl=30)

PanicException: must be valid hashmap: PyErr { type: <class 'TypeError'>, value: TypeError("'int' object cannot be converted to 'PyString'"), traceback: None }

There is an away to set ttl on op.write?

[Xuanwo]

Hi, thanks for raising this. Please try:

import opendal
op = opendal.Operator('redis', host='localhost:6379', default_ttl='30')

We are working on new config design to make it better in the future.

[Xuanwo]

PanicException: config deserialize must succeed: Error("invalid type: string "30", expected struct Duration")

Oh, I got it. We perform a convert between rust Duration and python datetime.timedelta. Please try passing a timedetla instead.

[jairhenrique]

@Xuanwo similar error.

thread '<unnamed>' panicked at src/operator.rs:64:22:
must be valid hashmap: PyErr { type: <class 'TypeError'>, value: TypeError("'timedelta' object cannot be converted to 'PyString'"), traceback: None }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
---------------------------------------------------------------------------
PanicException                            Traceback (most recent call last)
Cell In[3], line 1
----> 1 op = opendal.Operator('redis', host='localhost:6379', default_ttl=timedelta(seconds=30))

PanicException: must be valid hashmap: PyErr { type: <class 'TypeError'>, value: TypeError("'timedelta' object cannot be converted to 'PyString'"), traceback: None }

[Xuanwo]

I will check out what happened here. Sorry for the wrong guidance.

[jairhenrique]

@Xuanwo correcting my comment about node, the same error happens there. I was passing the parameter with the wrong name.

thread '<unnamed>' panicked at /Users/runner/work/opendal/opendal/core/src/services/redis/backend.rs:200:14:
config deserialize must succeed: Error("invalid type: string \"1\", expected struct Duration")
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
fatal runtime error: failed to initiate panic, error 5
fish: Job 1, 'node index.js' terminated by signal SIGABRT (Abort)

[jairhenrique]

Looking at the implementation of memcached, default_ttl expects Duration as well and perhaps the same issue should occur.