Skip to content

Commit

Permalink
fix conflicts and optimize cloud refresher
Browse files Browse the repository at this point in the history
  • Loading branch information
yanhom1314 committed Dec 16, 2023
1 parent 633d9ca commit 1f803cd
Show file tree
Hide file tree
Showing 10 changed files with 29 additions and 26 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -67,9 +67,10 @@ protected void initialize() {
super.initialize();

val bean = JVMTI.getInstance(StarlightServer.class);
if (!(bean instanceof DefaultStarlightServer starlightServer)) {
if (!(bean instanceof DefaultStarlightServer)) {
return;
}
val starlightServer = (DefaultStarlightServer) bean;
val uri = (URI) ReflectionUtil.getFieldValue(DefaultStarlightServer.class, URI_FIELD, starlightServer);
val serverPeer = (ServerPeer) ReflectionUtil.getFieldValue(DefaultStarlightServer.class,
SERVER_PEER_FIELD, starlightServer);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,12 +58,12 @@ public boolean supportsEventType(ResolvableType resolvableType) {
@Override
public void onApplicationEvent(@NonNull ApplicationEvent event) {
try {
if (event instanceof RefreshEvent refreshEvent) {
doRefresh(refreshEvent.getDtpProperties());
} else if (event instanceof CollectEvent collectEvent) {
doCollect(collectEvent.getDtpProperties());
} else if (event instanceof AlarmCheckEvent alarmCheckEvent) {
doAlarmCheck(alarmCheckEvent.getDtpProperties());
if (event instanceof RefreshEvent) {
doRefresh(((RefreshEvent) event).getDtpProperties());
} else if (event instanceof CollectEvent) {
doCollect(((CollectEvent) event).getDtpProperties());
} else if (event instanceof AlarmCheckEvent) {
doAlarmCheck(((AlarmCheckEvent) event).getDtpProperties());
}
} catch (Exception e) {
log.error("DynamicTp adapter, event handle failed.", e);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,19 +68,19 @@ protected void initialize() {
String key = Optional.ofNullable(internalServer)
.map(server -> {
final SocketAddress address = server.getListenSocketAddress();
if (address instanceof InetSocketAddress inetSocketAddress) {
return String.valueOf(inetSocketAddress.getPort());
} else if (address instanceof InProcessSocketAddress inProcessSocketAddress) {
return inProcessSocketAddress.getName();
if (address instanceof InetSocketAddress) {
return String.valueOf(((InetSocketAddress) address).getPort());
} else if (address instanceof InProcessSocketAddress) {
return ((InProcessSocketAddress) address).getName();
}
return null;
}).orElse(null);
if (Objects.isNull(key)) {
continue;
}
val executor = (Executor) ReflectionUtil.getFieldValue(ServerImpl.class, EXECUTOR_FIELD, serverImpl);
if (Objects.nonNull(executor) && executor instanceof ThreadPoolExecutor threadPoolExecutor) {
enhanceOriginExecutor(genTpName(key), threadPoolExecutor, EXECUTOR_FIELD, serverImpl);
if (Objects.nonNull(executor) && executor instanceof ThreadPoolExecutor) {
enhanceOriginExecutor(genTpName(key), (ThreadPoolExecutor) executor, EXECUTOR_FIELD, serverImpl);
}
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,8 @@ protected void initialize() {
}
beans.forEach((k, v) -> {
val executor = v.dispatcher().executorService();
if (executor instanceof ThreadPoolExecutor threadPoolExecutor) {
enhanceOriginExecutor(genTpName(k), threadPoolExecutor, EXECUTOR_SERVICE_FIELD, v.dispatcher());
if (executor instanceof ThreadPoolExecutor) {
enhanceOriginExecutor(genTpName(k), (ThreadPoolExecutor) executor, EXECUTOR_SERVICE_FIELD, v.dispatcher());
}
});
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -81,14 +81,16 @@ protected void initialize() {
servers.forEach(v -> {
ThreadPoolExecutor executor = null;
ServerConfig serverConfig = null;
if (v instanceof BoltServer boltServer) {
executor = boltServer.getBizThreadPool();
if (v instanceof BoltServer) {
BoltServer server = (BoltServer) v;
executor = server.getBizThreadPool();
serverConfig = (ServerConfig) ReflectionUtil.getFieldValue(BoltServer.class,
SERVER_CONFIG_FIELD, boltServer);
} else if (v instanceof AbstractHttpServer httpServer) {
executor = httpServer.getBizThreadPool();
SERVER_CONFIG_FIELD, server);
} else if (v instanceof AbstractHttpServer) {
AbstractHttpServer server = (AbstractHttpServer) v;
executor = server.getBizThreadPool();
serverConfig = (ServerConfig) ReflectionUtil.getFieldValue(AbstractHttpServer.class,
SERVER_CONFIG_FIELD, httpServer);
SERVER_CONFIG_FIELD, server);
}
if (Objects.isNull(executor) || Objects.isNull(serverConfig)) {
return;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public boolean supportsEventType(@NonNull Class<? extends ApplicationEvent> even

@Override
public void onApplicationEvent(@NonNull ApplicationEvent event) {
if (event instanceof EnvironmentChangeEvent ece && needRefresh(ece.getKeys())) {
if (needRefresh(((EnvironmentChangeEvent) event).getKeys())) {
refresh(environment);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public boolean supportsEventType(@NonNull Class<? extends ApplicationEvent> even

@Override
public void onApplicationEvent(@NonNull ApplicationEvent event) {
if (event instanceof EnvironmentChangeEvent ece && needRefresh(ece.getKeys())) {
if (needRefresh(((EnvironmentChangeEvent) event).getKeys())) {
refresh(environment);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public boolean supportsEventType(@NonNull Class<? extends ApplicationEvent> even

@Override
public void onApplicationEvent(@NonNull ApplicationEvent event) {
if (event instanceof EnvironmentChangeEvent ece && needRefresh(ece.getKeys())) {
if (needRefresh(((EnvironmentChangeEvent) event).getKeys())) {
refresh(environment);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public boolean supportsEventType(@NonNull Class<? extends ApplicationEvent> even

@Override
public void onApplicationEvent(@NonNull ApplicationEvent event) {
if (event instanceof EnvironmentChangeEvent ece && needRefresh(ece.getKeys())) {
if (needRefresh(((EnvironmentChangeEvent) event).getKeys())) {
refresh(environment);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public boolean supportsEventType(@NonNull Class<? extends ApplicationEvent> even

@Override
public void onApplicationEvent(@NonNull ApplicationEvent event) {
if (event instanceof EnvironmentChangeEvent ece && needRefresh(ece.getKeys())) {
if (needRefresh(((EnvironmentChangeEvent) event).getKeys())) {
refresh(environment);
}
}
Expand Down

0 comments on commit 1f803cd

Please sign in to comment.