Huge speedup (factor 30+) for update_receiver and update_device due to multi-column indices

ogn/model/aircraft_beacon.py

@@ -1,4 +1,4 @@
-from sqlalchemy import Column, String, Integer, Float, Boolean, SmallInteger, ForeignKey
+from sqlalchemy import Column, String, Integer, Float, Boolean, SmallInteger, ForeignKey, Index
 from sqlalchemy.orm import relationship
 
 from .beacon import Beacon
@@ -42,12 +42,16 @@ class AircraftBeacon(Beacon):
     distance = Column(Float)
 
     # Relations
-    receiver_id = Column(Integer, ForeignKey('receiver.id', ondelete='SET NULL'), index=True)
+    receiver_id = Column(Integer, ForeignKey('receiver.id', ondelete='SET NULL'))
     receiver = relationship('Receiver', foreign_keys=[receiver_id])
 
-    device_id = Column(Integer, ForeignKey('device.id', ondelete='SET NULL'), index=True)
+    device_id = Column(Integer, ForeignKey('device.id', ondelete='SET NULL'))
     device = relationship('Device', foreign_keys=[device_id])
 
+    # Multi-column indices
+    Index('ix_aircraft_beacon_receiver_id_receiver_name', 'receiver_id', 'receiver_name')
+    Index('ix_aircraft_beacon_device_id_address', 'device_id', 'address')
+
     def __repr__(self):
         return "<AircraftBeacon %s: %s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s>" % (
             self.address_type,

ogn/model/receiver_beacon.py

@@ -1,4 +1,4 @@
-from sqlalchemy import Column, Float, String, Integer, SmallInteger, ForeignKey
+from sqlalchemy import Column, Float, String, Integer, SmallInteger, ForeignKey, Index
 from sqlalchemy.orm import relationship
 
 from .beacon import Beacon
@@ -34,9 +34,12 @@ class ReceiverBeacon(Beacon):
     status = Column(SmallInteger, index=True)
 
     # Relations
-    receiver_id = Column(Integer, ForeignKey('receiver.id', ondelete='SET NULL'), index=True)
+    receiver_id = Column(Integer, ForeignKey('receiver.id', ondelete='SET NULL'))
     receiver = relationship('Receiver', foreign_keys=[receiver_id])
 
+    # Multi-column indices
+    Index('ix_receiver_beacon_receiver_id_name', 'receiver_id', 'name')
+
     def __repr__(self):
         return "<ReceiverBeacon %s: %s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s>" % (
             self.version,

tests/model/all_classes.py

@@ -15,5 +15,6 @@ class TestStringMethods(unittest.TestCase):
         except AttributeError as e:
             raise AssertionError(e)
 
+
 if __name__ == '__main__':
     unittest.main()