refactor(connection): remove detachSub

[ngjaying]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 89.91228% with 23 lines in your changes missing coverage. Please review.

Project coverage is 71.11%. Comparing base (7248c66) to head (f1b34df).
Report is 3 commits behind head on master.

Files with missing lines	Patch %	Lines
pkg/connection/pool.go	93.28%	6 Missing and 3 partials ⚠️
pkg/connection/mock.go	42.86%	4 Missing ⚠️
extensions/impl/sql/sink.go	0.00%	3 Missing ⚠️
internal/io/http/httpserver/httpconn.go	40.00%	3 Missing ⚠️
internal/io/http/httpserver/websocketConn.go	50.00%	2 Missing ⚠️
internal/io/mqtt/source.go	50.00%	1 Missing ⚠️
internal/server/server.go	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3172      +/-   ##
==========================================
- Coverage   71.13%   71.11%   -0.02%     
==========================================
  Files         385      386       +1     
  Lines       43187    43210      +23     
==========================================
+ Hits        30721    30728       +7     
- Misses      10033    10047      +14     
- Partials     2433     2435       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.