sandynz commented on code in PR #29810:
URL: https://github.com/apache/shardingsphere/pull/29810#discussion_r1462986538
##########
kernel/data-pipeline/core/src/main/java/org/apache/shardingsphere/data/pipeline/core/listener/PipelineContextManagerLifecycleListener.java:
##########
@@ -64,22 +64,22 @@ public void onInitialized(final String databaseName, final
ContextManager contex
ElasticJobServiceLoader.registerTypedService(ElasticJobListener.class);
try {
dispatchEnablePipelineJobStartEvent(contextKey);
- } catch (final IllegalArgumentException | PipelineJobNotFoundException
ex) {
+ } catch (final IllegalArgumentException |
UnsupportedOperationException | PipelineJobNotFoundException ex) {
log.error("Dispatch enable pipeline job start event failed", ex);
}
}
private void dispatchEnablePipelineJobStartEvent(final PipelineContextKey
contextKey) {
JobConfigurationAPI jobConfigAPI =
PipelineAPIFactory.getJobConfigurationAPI(contextKey);
List<JobBriefInfo> allJobsBriefInfo =
PipelineAPIFactory.getJobStatisticsAPI(contextKey).getAllJobsBriefInfo()
- .stream().filter(each ->
!each.getJobName().startsWith("_")).collect(Collectors.toList());
+ .stream().filter(each -> !each.getJobName().startsWith("_") &&
!each.getJobName().startsWith("j02")).collect(Collectors.toList());
Review Comment:
It's better not hard-code job name prefix
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
