Feature/157 connector refactor

[edigonzales]

@dkubanyi @Saela Das dünkt mich jetzt nicht wie angedacht. Die Idee war doch, dass wir die alte API erhalten können [dburi, dbuser, dbpass]. Jetzt gibt es drei einzelne Properties?

[dkubanyi]

Ach so, ich dachte die database variable soll entfernt werden. Ich werde es korrigieren

[dkubanyi]

@Saela @edigonzales erledigt

[edigonzales]

@dkubanyi @Saela Vielleicht habe ich es falsch verstanden. Ich sehe, dass es ja bereits ein TaskUtil gibt, d.h. der Code gibt es nur einmal (und nicht in jedem Task) https://github.com/sogis/gretl/blob/gretl_3.0/gretl/src/main/java/ch/so/agi/gretl/util/TaskUtil.java#L44

Warum jetzt noch der Weg über die DatabaseTask-Klasse?

[dkubanyi]

@edigonzales möchtest du lieber die Utils Methode verwenden? Ich dachte es wäre übersichtlicher wenn es für jede Task die Datenbankverbindung braucht eine Baseklasse gäbe. Ich persönlich würde dafür lieber die Klasse verwenden, aber wenn du die Utils Methode willst, werde ich es ändern.

[edigonzales]

@dkubanyi Es gibt auch noch die Db2Db-Klasse, die zwei Connectoren hat. Und die ili2pg-Klassen, die auch einen Connector benutzen. Von daher denke ich, dass es einfacher ist, wenn man mit der Utils-Methode arbeitet.

[sonarcloud[bot]]

Quality Gate failed

Failed conditions
31.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

[dkubanyi]

@edigonzales Fertig. Sonarcloud schlägt fehl, da es die gesamte Codebase prüft, nicht nur die Änderungen in diesem PR. Vielleicht können wir es so konfigurieren dass es nur Änderungen überprüft? Edit: Es prüft jede geänderte File, nicht nur Zeilen