global: suppress warnings during app instantiation

[jacquerie]

Currently inspirehep shell prints the following output when DEBUG=True:

/Users/jacquerie/.virtualenvs/inspire/lib/python2.7/site-packages/flask/exthook.py:71: ExtDeprecationWarning: Importing flask.ext.cache is deprecated, use flask_cache instead.
  .format(x=modname), ExtDeprecationWarning

/Users/jacquerie/.virtualenvs/inspire/lib/python2.7/site-packages/sqlalchemy/ext/declarative/clsregistry.py:120: SAWarning: This declarative base already contains a class with the same class name and module name as flask_sqlalchemy.RecordMetadataVersion, and will be replaced in the string-lookup table.
  item.__name__

/Users/jacquerie/.virtualenvs/inspire/lib/python2.7/site-packages/sqlalchemy/orm/properties.py:194: SAWarning: On mapper Mapper|RecordMetadataVersion|records_metadata_version, primary key column 'records_metadata_version.transaction_id' is being combined with distinct primary key column 'records_metadata_version.transaction_id' in attribute 'transaction_id'.  Use explicit properties to give each column its own mapped attribute name.
  self.columns[0], self.key))

Python 2.7.12 (default, Oct 11 2016, 05:20:59)
[GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.38)] on darwin
IPython: 5.1.0
App: inspirehep [debug]
Instance: /Users/jacquerie/.virtualenvs/inspire/bin/../var/inspirehep-instance

In [1]:

which gets even more annoying when you are executing several inspirehep commands in a row, like in scripts/clean_assets.

• The first three flask.ext ones come from https://github.com/inveniosoftware/invenio-oauthclient, and will be fixed by global: ext warning fix inveniosoftware/invenio-oauthclient#106.
• I don't know what's causing the fourth one.
• The next flask.ext one comes from https://github.com/thadeusb/flask-cache, and there's little chance of getting it fixed upstream (New release? Is anybody managing this anymore? thadeusb/flask-cache#151) unless we switch to https://github.com/sh4nks/flask-caching.
• The last two come from SQLAlchemy-Continuum. I'm not sure where, but note that this warning is happening upstream as well, so maybe the culprit is not in INSPIRE: https://travis-ci.org/inveniosoftware/invenio-records-rest/jobs/166397399#L591.

[jacquerie]

We have a few more:

/Users/jacquerie/.virtualenvs/inspire/lib/python2.7/site-packages/flask/exthook.py:71: ExtDeprecationWarning: Importing flask.ext.admin is deprecated, use flask_admin instead.
  .format(x=modname), ExtDeprecationWarning

/Users/jacquerie/.virtualenvs/inspire/lib/python2.7/site-packages/flask/exthook.py:71: ExtDeprecationWarning: Importing flask.ext.admin.contrib is deprecated, use flask_admin.contrib instead.
  .format(x=modname), ExtDeprecationWarning

/Users/jacquerie/.virtualenvs/inspire/lib/python2.7/site-packages/flask/exthook.py:71: ExtDeprecationWarning: Importing flask.ext.admin.contrib.sqla is deprecated, use flask_admin.contrib.sqla instead.
  .format(x=modname), ExtDeprecationWarning

/Users/jacquerie/.virtualenvs/inspire/lib/python2.7/site-packages/sqlalchemy/util/langhelpers.py:754: SADeprecationWarning: Use .target
  obj.__dict__[self.__name__] = result = self.fget(obj)

First three will be fixed by inveniosoftware/invenio-oauthclient#106, fourth one is harder.

[jacquerie]

Fourth one is caused by Flask-Admin, specifically this line: https://github.com/flask-admin/flask-admin/blob/1fa4ba80052910fed4f80bc339b0f04955233ed4/flask_admin/contrib/sqla/view.py#L597. I submitted a PR for it: pallets-eco/flask-admin#1447.

[jacquerie]

Yet another squashed warning: inveniosoftware/invenio-rest#67.

[jacquerie]

Another small one will be fixed by inveniosoftware/invenio-accounts#229.

[jacquerie]

And another small one by inveniosoftware/invenio-accounts#245.

[jacquerie]

The last two come from SQLAlchemy-Continuum. I'm not sure where, but note that this warning is happening upstream as well, so maybe the culprit is not in INSPIRE: https://travis-ci.org/inveniosoftware/invenio-records-rest/jobs/166397399#L591.

I now know that this comes from invenio-db: the logic in https://github.com/inveniosoftware/invenio-db/blob/5e387bbbbd494553af4ff960dd09941cd4dec9ac/invenio_db/ext.py#L104-L156 might be instantiating the records_metadata table more times than necessary.

[jacquerie]

I now know that this comes from invenio-db: the logic in https://github.com/inveniosoftware/invenio-db/blob/5e387bbbbd494553af4ff960dd09941cd4dec9ac/invenio_db/ext.py#L104-L156 might be instantiating the records_metadata table more times than necessary.

Tracked (and explained) in inveniosoftware/invenio-db#96.

[jacquerie]

Tracked (and explained) in inveniosoftware/invenio-db#96.

...and finally solved by @slint, who will get a 🍺 next time I see him.

[jacquerie]

Since all these warnings are now fixed in commits that are at worst pending a new release of the libraries they are in, we can close this.