Stop creating indexes each time a document is inserted (#62)

* Stop creating indexes each time a document is inserted

We shouldn't treat the underlying database as something we own but
rather see this application as one of many application that interact
with an Eiffel event store. From that point of view it doesn't make
sense to create indexes each time a document is created.

Although the fields we previously created indexes for should be indexed
in pretty much any deployment most deployments will also want to index
additional fields. Adding a configuration option to manage indexes in
this application doesn't feel right, nor does spreading out index
management in multiple places.

Hence, remove index creations from insert_to_db(). This should also
improve performance.

* Fix formatting to please Black

Author: magnusbaeck

src/eiffel_graphql_api/graphql/db/database.py

@@ -115,15 +115,11 @@ def insert_to_db(event, _=None):
     try:
         database = get_database()
         collection = database[event.meta.type]
-        collection.create_index([("meta.id", pymongo.ASCENDING)])
-        collection.create_index([("links.target", pymongo.ASCENDING)])
         doc = event.json
         doc["_id"] = event.meta.event_id
         collection.insert_one(doc)
     except pymongo.errors.DuplicateKeyError:
-        LOGGER.warning(
-            "Event already exists in the database, " "skipping: %r", event.json
-        )
+        LOGGER.warning("Event already exists in the database, skipping: %r", event.json)
         return True
     except pymongo.errors.ConnectionFailure as exception:
         LOGGER.warning("%r", exception)