Comparing datetime about walrus

[coleifer]

What field class are you using for last_seen?

[luisdemarchi]

@coleifer Following is the actual structure

# Presence REDIS DB (no-SQL)
db = walrus.Database(host='localhost', port=6379, db=0)


class Presence(walrus.Model):
    database = db
    channel_name = walrus.TextField(primary_key=True)
    user = walrus.TextField(fts=True, index=True)
    last_seen = walrus.DateTimeField(default=datetime.datetime.now)
    is_accessible = walrus.BooleanField(index=True, default=False)

[coleifer]

Aha. The problem is the way you are associating the Presence model with the db object.

You should use this instead:

class Presence(walrus.Model):
    __database__ = db  # Double underscores around "database"
    channel_name ... etc.

[luisdemarchi]

@coleifer Sorry, but it's in the documentation if this parameter is not set will take the default. So if I'm wrong the variable should pick the default. I made this change and got several different errors in each of the queries that worked before.

PS: Remembering that I am from Brazil and I am supported by the translator.

>>> Presence.create(channel_name="asdfhkasdlfkjashfdaskjhfsakjhdfa")
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "/Volumes/MacHD/Users/Git/playboy-connect-webservice/.env/lib/python3.6/site-packages/walrus/models.py", line 756, in create
    instance.save()
  File "/Volumes/MacHD/Users/Git/playboy-connect-webservice/.env/lib/python3.6/site-packages/walrus/models.py", line 942, in save
    self.delete(for_update=True)
  File "/Volumes/MacHD/Users/Git/playboy-connect-webservice/.env/lib/python3.6/site-packages/walrus/models.py", line 906, in delete
    original_instance = self.load(hash_key, convert_key=False)
  File "/Volumes/MacHD/Users/Git/playboy-connect-webservice/.env/lib/python3.6/site-packages/walrus/models.py", line 877, in load
    if not cls.database.hash_exists(primary_key):
AttributeError: 'NoneType' object has no attribute 'hash_exists'


>>> for presence in Presence.all():
...     print(presence.last_seen)
...     
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "/Volumes/MacHD/Users/Git/playboy-connect-webservice/.env/lib/python3.6/site-packages/walrus/models.py", line 777, in all
    for result in cls._query.all_index():
  File "/Volumes/MacHD/Users/Git/playboy-connect-webservice/.env/lib/python3.6/site-packages/walrus/models.py", line 374, in all_index
    return self.model_class.database.Set(self.make_key('all'))
AttributeError: 'NoneType' object has no attribute 'Set'

It is noteworthy that before I could read and write, I just could not return this query. So I find it strange to be a problem connecting to the bank.

[coleifer]

@luisdemarchi -- I think you must be running an older version of walrus. Can you please try upgrading? Depending on how you installed it, you can probably run pip install -U walrus.

[luisdemarchi]

@coleifer:

pip install -U walrus
Requirement already up-to-date: walrus in ./.env/lib/python3.6/site-packages
Requirement already up-to-date: redis in ./.env/lib/python3.6/site-packages (from walrus)

[coleifer]

I'm sorry -- I thought that I had released those changes (database -> __database__) but it turns out that they're just sitting in the master branch right now. So your code is correct for the current release. Reopening this to look into it some more.

[coleifer]

A-ha, I know what the issue is! You need to specify index=True to enable filtering using a field. In your case, the last_seen field needs to have an index.

class Presence(walrus.Model):
    __database__ = db  # If 0.4.0 or newer, use double-underscores.
    channel_name = walrus.TextField(primary_key=True)
    user = walrus.TextField(fts=True, index=True)
    last_seen = walrus.DateTimeField(default=datetime.datetime.now, index=True)  # Add "index=True".
    is_accessible = walrus.BooleanField(index=True, default=False)

Docs: http://walrus.readthedocs.io/en/latest/models.html#filtering-records

[luisdemarchi]

@coleifer Wow! Sorry for this grotesque mistake, this requirement is very clear in its documentation. Thanks a lot for the help. Once the app is ready and published, if possible I'll let you know;)

[coleifer]

Not a problem, I'm sorry I didn't notice it sooner myself. Glad it's working now.