-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ability to handle import logs from multiple projects #5511
Conversation
Build # 2970 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2970/ to view the results. |
eventBus.addHandler(WsAgentStateEvent.TYPE, new WsAgentStateHandler() { | ||
@Override | ||
public void onWsAgentStarted(WsAgentStateEvent event) { | ||
transmitter.newRequest().endpointId(WS_AGENT_ENDPOINT).methodName(EVENT_IMPORT_OUTPUT_SUBSCRIBE).noParams().sendAndSkipResult(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
formatting issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/2975/ |
@Singleton | ||
public class ProjectImportNotificationSubscriber { | ||
|
||
public static final String WS_AGENT_ENDPOINT = "ws-agent"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
may this property be defined at one place similar to AppContext one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's an endpoint id which is not stored in application context
* Ability to handle import logs from multiple projects * Formatting
What does this PR do?
Add ability to transmit importer logs from multiple project at the same time.
What issues does this PR fix or reference?
#5428
Changelog
Add ability to transmit importer logs from multiple project at the same time
Release Notes
N/A
Docs PR
N/A