Change Details

This class has more than 6000 lines and more than 100 public functions, it does everything related to databases and needs to be broken down. Some ideas that were brought up by @daniel are: - Migrate transaction management to its own class and interface - Migrate SQL building blocks to SQLFlavor or something similar - This requires finding a way to pass addQuote to that class, we can introduce an interface like DatabaseQuoteBuilder (better name welcome) and pass self in constructor. - Move replication-based stuff to its own class, e.g. getLag() - Remove legacy cruft from introduction of ResultWrapper: T286694

This class has more than 6000 lines and more than 100 public functions, it does everything related to databases and needs to be broken down. Some ideas that were brought up by @daniel are: - Migrate transaction management to its own class and interface - Migrate SQL building blocks to SQLFlavor or something similar - This requires finding a way to pass addQuote to that class, we can introduce an interface like DatabaseQuoteBuilder (better name welcome) and pass self in constructor. - Move replication-based stuff to its own class, e.g. getLag() - Remove legacy cruft from introduction of ResultWrapper: T286694 - Move schema update and related functions to IMaintainableDatabase (T190396)

This class has more than 6000 lines and more than 100 public functions, it does everything related to databases and needs to be broken down. Some ideas that were brought up by @daniel are: - Migrate transaction management to its own class and interface - Migrate SQL building blocks to SQLFlavor or something similar - This requires finding a way to pass addQuote to that class, we can introduce an interface like DatabaseQuoteBuilder (better name welcome) and pass self in constructor. - Move replication-based stuff to its own class, e.g. getLag() - Remove legacy cruft from introduction of ResultWrapper: T286694 - Move schema update and related functions to IMaintainableDatabase (T190396)