Tomo-9925 / cnet

Controlling and logging communication of process in Docker container
2 stars 0 forks source link

処理の並列化 #37

Closed Tomo-9925 closed 3 years ago

Tomo-9925 commented 3 years ago

変更内容

masibw commented 3 years ago

書き忘れてたんですが handlerをhandlerパッケージにまとめた方が良い気がします

Tomo-9925 commented 3 years ago

mainが肥大化しすぎているのでhandlerでパッケージを分けます. あと,containerがレシーバで書くべき内容が含まれているのに,変にDocker向けの関数は別に分けようとしているので,パッケージで分けてレシーバに変更します(語彙力無し,多分コードで見てもらったほうが良い).

Tomo-9925 commented 3 years ago

よくよく考えると,ディレクトリ構造の変更は別ブランチのほうが良かったですね…

この変更によって1つ気に入らないことができました.procCheckSocketAndCommunicatedContainerCheckSocketAndCommunicatedDockerContainerになりました… (思い出したけど,procの中にもdockerdのDNSの通信ををスルーするルールあるやん…) レシーバがついてるとはいえ,同じcontainer.Containersの構造なのであれば,うまいことメモリーコピーなしにListを取り出せないものだろうか…

procもなかなかにエグいので,落ち着いたら変更します.