Edge-home-orchestration-go: [MNEDC] Refactoring in Bezug auf Architektur

Erstellt am 16. Nov. 2020  ·  5Kommentare  ·  Quelle: lf-edge/edge-home-orchestration-go

Ich denke, dass die mnedcmgr-Funktion unter Discovermgr stehen sollte, da sie in Bezug auf die Funktionalität ähnlich sind.
Wie wäre es, den mnedcmgr-Code unter den discoverymgr zu verschieben?
Unten sollte geändert werden.

  • [x] Ändern Sie den Paketnamen von mnedcmgr in mnedc oder so.
  • [ ] Der Discoverymgr sollte StartMNEDCServer() und StartMNEDCClient() aufrufen.

Alle Kommentare sind willkommen!

question refactoring

Hilfreichster Kommentar

@t25kim @MoonkiHong : Wir haben diese Modifikation überprüft. Scheint einen zyklischen Import zu beinhalten, der von Go nicht unterstützt wird. Also prüfen wir, ob wir dasselbe ändern und testen.

Alle 5 Kommentare

@ayush-kr @suresh-lc Wie steht es um Ihre Meinung (aus Sicht der Beitragenden)?

Ich denke, dass die mnedcmgr-Funktion unter Discovermgr stehen sollte, da sie in Bezug auf die Funktionalität ähnlich sind.
Wie wäre es, den mnedcmgr-Code unter den discoverymgr zu verschieben?
Unten sollte geändert werden.

  • [ ] Ändern Sie den Paketnamen von mnedcmgr in mnedc oder so.
  • [ ] Der Discoverymgr sollte StartMNEDCServer() und StartMNEDCClient() aufrufen.

Alle Kommentare sind willkommen!

@t25kim Vielen Dank für einen tollen Vorschlag. Wir werden abschätzen, ob es möglich ist, die Funktionen von mnedcmgr in discoverymgr mit ungefähr Mitwirkenden von mnedc zu migrieren.

@t25kim @MoonkiHong : Wir haben diese Modifikation überprüft. Scheint einen zyklischen Import zu beinhalten, der von Go nicht unterstützt wird. Also prüfen wir, ob wir dasselbe ändern und testen.

@t25kim @MoonkiHong : Wir haben diese Modifikation überprüft. Scheint einen zyklischen Import zu beinhalten, der von Go nicht unterstützt wird. Also prüfen wir, ob wir dasselbe ändern und testen.

@suresh-lc Vielen Dank für Ihre prompte Analyse. Wie Sie vorgeschlagen haben, lassen Sie uns weiter über den besten Ansatz sprechen, wenn Sie die Überprüfung zum Ändern und Testen abgeschlossen haben, insbesondere über den _zyklischen Import, der von GoLang nicht unterstützt wird_. Wir warten auf Ihre Updates. ^^

Dieses Problem ist immer noch gültig, da die letzte zusammengeführte PR #178 Change package name from mnedcmgr to mnedc or something. behoben hat.

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen

Verwandte Themen

t25kim picture t25kim  ·  5Kommentare

MoonkiHong picture MoonkiHong  ·  4Kommentare

t25kim picture t25kim  ·  5Kommentare

t25kim picture t25kim  ·  3Kommentare

t25kim picture t25kim  ·  3Kommentare