[ISSUE #4838] Deprecate unused eventMesh.connector.plugin.type etc. properties

[Pil0tXia]

Fixes #4838

Modifications

• Deprecate eventMesh.connector.plugin.type: EventMesh does not use this config to determine which connector to use. This config can easily confuse users. It is not referenced in implementation code either.
• Deprecate registerIntervalInMills, fetchRegistryAddrIntervalInMills: These two configs were placed seperatly from metaStorage plugins, however they are always (in Git history) misconfigured (missing plugin level) & not used since EventMesh initiated.
  

  eventmesh/eventmesh-runtime/conf/eventmesh.properties

  Lines 63 to 64 in fe2bd7c

  	eventMesh.server.metaStorage.metaStorageIntervalInMills=10000
  	eventMesh.server.metaStorage.fetchMetaStorageAddrIntervalInMills=20000

  
  

  eventmesh/eventmesh-common/src/main/java/org/apache/eventmesh/common/config/CommonConfiguration.java

  Lines 70 to 74 in fe2bd7c

  	@ConfigField(field = "metaStorage.plugin.metaStorageIntervalInMills")
  	private Integer eventMeshMetaStorageIntervalInMills = 10 * 1000;
  	@ConfigField(field = "metaStorage.plugin.fetchMetaStorageAddrIntervalInMills")
  	private Integer eventMeshFetchMetaStorageAddrInterval = 10 * 1000;

• Deprecate eventMesh.server.defibus.client.comsumeTimeoutInMin and 'defibus' related usages: These usages are commented and not used since EventMesh initiated, and may output one line of null log.
  
• Place similar properties together and add a seperator.
• I found out that I forgot to unify null != object in [ISSUE #4808] Unify all null == object usage to standard object == null #4809 when writting this PR (only null == object usages unified), so this PR contains supplements of the previous one.

Documentation

• Does this pull request introduce a new feature? (no)

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 12.00000% with 22 lines in your changes are missing coverage. Please review.

Project coverage is 16.18%. Comparing base (e04156b) to head (6c63240).
Report is 6 commits behind head on master.

❗ Current head 6c63240 differs from pull request most recent head c477f0d. Consider uploading reports for the commit c477f0d to get more accurate results

Files	Patch %	Lines
.../jdbc/source/dialect/cdc/mysql/MysqlCdcEngine.java	0.00%	3 Missing ⚠️
...time/core/protocol/tcp/client/session/Session.java	0.00%	3 Missing ⚠️
...pache/eventmesh/common/EventMeshThreadFactory.java	0.00%	0 Missing and 2 partials ⚠️
...ventmesh/connector/lark/sink/ImServiceHandler.java	0.00%	0 Missing and 2 partials ⚠️
...che/eventmesh/runtime/boot/AbstractHTTPServer.java	0.00%	2 Missing ⚠️
...ntmesh/storage/rabbitmq/client/RabbitmqClient.java	0.00%	0 Missing and 2 partials ⚠️
...r/jdbc/dialect/AbstractGeneralDatabaseDialect.java	0.00%	1 Missing ⚠️
...tlr4/mysql/listener/CreateTableParserListener.java	0.00%	1 Missing ⚠️
.../eventmesh/connector/jdbc/table/catalog/Table.java	0.00%	1 Missing ⚠️
...entmesh/meta/consul/service/ConsulMetaService.java	0.00%	0 Missing and 1 partial ⚠️
... and 4 more

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #4839      +/-   ##
============================================
- Coverage     16.20%   16.18%   -0.03%     
+ Complexity     1710     1708       -2     
============================================
  Files           857      858       +1     
  Lines         30883    30863      -20     
  Branches       2685     2691       +6     
============================================
- Hits           5005     4994      -11     
+ Misses        25410    25400      -10     
- Partials        468      469       +1     

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

[mxsm]

LGTM