revert at least partially https://github.com/powa-team/powa/issues/63

powa-team / powa-archivist

powa-archivist: the powa PostgreSQL extension

http://powa.readthedocs.io/

PostgreSQL License

51 stars 20 forks source link

revert at least partially https://github.com/powa-team/powa/issues/63 #18

Closed marco44 closed 4 years ago

marco44 commented 5 years ago

This was mostly a mistake. We cannot do a lot with this historical data anyway, and it makes things confusing.

The plan is:

Still maintain the powa_databases table the same way
Expire the history records the same they are today (using retention)
Expire the powa_databases and linked powa_statements for the dbid during the coalesce, deleting the database effectively from history.

marco44 commented 5 years ago

For powa-web, go check framework.py, def databases()

rjuju commented 4 years ago

@marco44 I don't recall exactly what has been done on that part. I see that powa_databases_purge() you introduced in https://github.com/powa-team/powa-archivist/pull/19 does some cleanup. Is there anything left to do regarding this issue?

marco44 commented 4 years ago

Don't remember either :)

rjuju commented 4 years ago

Damn :) Well, I'll close this issue for now as it looks fixed, if you hit again any issue with dropped database we'll try to actually fix it this time!