93_DbLog - Vorschläge zur Weiterentwicklung

[Tobias]

na der Patch von vorheriger Seite... Contrib nehme ich gerne um größere Änderungen vorher durch andere prüfen zu lassen ohne gleich auf Alle loszulassen. Nach 2-3 Wochen kommt es auch contrib wieder raus ins Main

[betateilchen]

Du wolltest von mir gestern einen patch haben und nun soll ich den morgen nochmal bauen.

Die Änderung bezüglich des wiederholten Verbindungsaufbau habe ich schon vor Wochen hier vorgeschlagen, ohne dass irgendwo aufgetaucht wäre - weder in contrib, noch in FHEM.

Wie oft eigentlich noch? Ich hab auch noch wichtigere FHEM Baustellen, als nun zum dritten Mal die gleiche Änderung zu diffen.

*FRUST*

[Tobias]

iss ja gut
ich pass ihn nächste Woche manuell an... hatte da nämlich schon andere Änderungend rin...

[betateilchen]

Irgendwann müssen wir das Modul trennen in den Daten-Teil und in den Datenbankverbindungs-Teil

Ich bau Dir den patch am Wochenende. Aber nur, wenn Du mir versprichst, dass ich es kein viertes Mal machen muss.

[betateilchen]

Ist denn die JETZT in SVN eingecheckte Version schon die aktuelle oder kommt da noch eine andere bis morgen?

[Tobias]

Hab den hotfix vorhin comitted. Mehr kommt nicht. Und versprochen. Den Patch Bau ich nächste Woche ein

Gesendet von meinem ALCATEL ONE TOUCH 997D mit Tapatalk

[betateilchen]

ok, ich baue also den Patch jetzt gegen diese Version:

# $Id: 93_DbLog.pm 5646 2014-04-25 18:03:29Z tobiasfaust $

[betateilchen]

Da hast Du das Gesamtkunstwerk...

[Tobias]

Da hast Du das Gesamtkunstwerk...

Danke... bekomme es nicht gepatcht, nur Hunks... ich machs dann doch von Hand. Musst nicht nochmal bauen. Dachte ich kann es heute morgen noch mal eben schnell machen...

[betateilchen]

Warum kannst Du nicht einfach mal genau beschreiben, was Du eigentlich willst?
Ich hatte doch extra gefragt, ob die gestrige Version nun die aktuelle ist - was willst Du denn jetzt schon wieder rumpatchen?

Hier hast Du die reine diff-Datei.

Du kannst mir auch Deine Entwicklungsdatei per email schicken, dann baue ich die Änderungen zur Verbindungsverwaltung dort ein.

[betateilchen]

Wenn Du es Dir einfach machen willst:

Nimm die folgende Modulteile aus meiner gestern hier angehängten kompletten Moduldatei und kopiere sie in "Deine" Version:

• sub DbLog_Define($@)
• sub _DbLog_readCfg($)
• sub DbLog_Connect($)
• sub DbLog_Set($@)

Der Rest ist Kleinkram und meist nur "Optik". Den kann ich Dir dann nachreichen, wenn wir mal eine KLARE Modulversion haben.

[Tobias]

sorry, war heute morgen zu früh und schon im Urlaubs-Packstress...Hatte das pm fälschlicherweise als diff angesehen. KANN natürlich nicht funktionieren...
Habe die komplette pm Datei getestet und im MainStream commited. Meine nächsten größeren Änderungen kommen dann ins contrib zum Testen.
Es kommen 3 neue Attribute (host, gueltig_bis, agg_kz), da weiß ich aber noch nicht wie ich das beim Define automatisiert prüfen und hinzufügen kann.

[betateilchen]

Es kommen 3 neue Attribute (host, gueltig_bis, agg_kz), da weiß ich aber noch nicht wie ich das beim Define automatisiert prüfen und hinzufügen kann.

So ein ähnliches Problem hab ich aktuell auch in der configDB.
Geh erstmal in Ruhe in Urlaub, vielleicht fällt mir ein sinnvoller Weg ein, wie wir in fhem bei Bedarf Datenbanken migrieren können.

[Tobias]

Naja, das Wort "migrieren" hat meiner (beruflichen) Erfahrung nach etwas negatives was mit viel Daten schaufeln assoziiert wird. Ich habe über 15Mio Datensätze in meiner produktiven DbLog DB... Da möchte ich nur Metadatenänderungen durchführen......  (Naja, gegenüber den 20TB in meinem Beruflichen Umfeld ist das nichts...)

[betateilchen]

Naja, solange Du nur zusätzliche Spalten anfügen willst, ist das nicht so dramatisch. Richtig kompliziert wird es, wenn man bestehende Felder verändern will (z.B. die Länge des Devicenamen von 32 auf 64 verändern) zumal, wenn das gewünschte Feld auch noch in einem Index verwendet wird.