Edge-home-orchestration-go: [MNEDC] Refatoração em termos de Arquitetura

Criado em 16 nov. 2020  ·  5Comentários  ·  Fonte: lf-edge/edge-home-orchestration-go

Eu acho que a função mnedcmgr deve estar sob o discoverymgr, pois eles são semelhantes no que diz respeito à funcionalidade.
Que tal mover o código mnedcmgr sob o discoverymgr?
Abaixo deve ser alterado.

  • [x] Mude o nome do pacote de mnedcmgr para mnedc ou algo assim.
  • [] O discoveryymgr deve chamar StartMNEDCServer () e StartMNEDCClient ().

Quaisquer comentários são bem-vindos!

question refactoring

Comentários muito úteis

@ t25kim @MoonkiHong : Verificamos essa modificação. Parece envolver importação cíclica que Go não suporta. Portanto, estamos verificando a alteração do mesmo e testando.

Todos 5 comentários

@ ayush-kr @ suresh-lc E a sua opinião (da perspectiva dos contribuidores)?

Eu acho que a função mnedcmgr deve estar sob o discoverymgr, pois eles são semelhantes no que diz respeito à funcionalidade.
Que tal mover o código mnedcmgr sob o discoverymgr?
Abaixo deve ser alterado.

  • [] Mude o nome do pacote de mnedcmgr para mnedc ou algo assim.
  • [] O discoveryymgr deve chamar StartMNEDCServer () e StartMNEDCClient ().

Quaisquer comentários são bem-vindos!

@ t25kim Obrigado por uma ótima sugestão. Vamos estimar sua viabilidade para migrar as funções de mnedcmgr para discoverymgr com relação aos contribuintes de mnedc .

@ t25kim @MoonkiHong : Verificamos essa modificação. Parece envolver importação cíclica que Go não suporta. Portanto, estamos verificando a alteração do mesmo e testando.

@ t25kim @MoonkiHong : Verificamos essa modificação. Parece envolver importação cíclica que Go não suporta. Portanto, estamos verificando a alteração do mesmo e testando.

@ suresh-lc Obrigado por sua pronta análise. Como você sugeriu, deixe-nos discutir mais sobre a melhor abordagem depois de concluir a verificação sobre como alterar o mesmo material e testar, especialmente sobre a _importação cíclica que GoLang não suporta_. Vamos esperar por suas atualizações. ^^

Este problema ainda é válido, pois o último PR # 178 mesclado resolveu Change package name from mnedcmgr to mnedc or something. .

Esta página foi útil?
0 / 5 - 0 avaliações