Skip to content

Conversation

@Alonexc
Copy link
Contributor

@Alonexc Alonexc commented Jul 20, 2023

Fixes #4245 .

Motivation

fix start failed

Modifications

fix start failed

Documentation

  • Does this pull request introduce a new feature? no

Copy link
Member

@mxsm mxsm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link

codecov bot commented Jul 20, 2023

Codecov Report

Merging #4258 (a278361) into master (4b994b0) will increase coverage by 0.01%.
The diff coverage is 0.00%.

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

@@             Coverage Diff              @@
##             master    #4258      +/-   ##
============================================
+ Coverage     16.75%   16.77%   +0.01%     
  Complexity     1415     1415              
============================================
  Files           592      592              
  Lines         26029    26032       +3     
  Branches       2384     2398      +14     
============================================
+ Hits           4362     4367       +5     
+ Misses        21230    21227       -3     
- Partials        437      438       +1     
Impacted Files Coverage Δ
...he/eventmesh/runtime/boot/EventMeshHTTPServer.java 0.52% <0.00%> (ø)
.../protocol/http/processor/CreateTopicProcessor.java 0.00% <0.00%> (ø)
.../protocol/http/processor/DeleteTopicProcessor.java 0.00% <0.00%> (ø)
...col/http/processor/QuerySubscriptionProcessor.java 0.00% <0.00%> (ø)
...ache/eventmesh/runtime/util/HttpResponseUtils.java 23.07% <0.00%> (+1.64%) ⬆️
...apache/eventmesh/runtime/util/ValueComparator.java 25.00% <ø> (ø)

... and 8 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@xwm1992 xwm1992 merged commit 022c051 into apache:master Jul 21, 2023
@Alonexc Alonexc deleted the fix_4245 branch July 22, 2023 11:03
@mdidde
Copy link

mdidde commented Sep 30, 2025

Seems like this issue still occurs with version 1.10.0 and 1.11.0 during docker start

java.lang.NullPointerException: traceServiceType: standalone is not supported
at java.util.Objects.requireNonNull(Objects.java:228) ~[?:1.8.0_342]
at org.apache.eventmesh.trace.api.TracePluginFactory.getEventMeshTraceService(TracePluginFactory.java:43) ~[eventmesh-trace-api-1.10.0-release.jar:1.10.0-release]
at org.apache.eventmesh.runtime.trace.Trace.traceBuilder(Trace.java:55) ~[eventmesh-runtime-1.10.0-release.jar:1.10.0-release]
at org.apache.eventmesh.runtime.trace.Trace.lambda$getInstance$0(Trace.java:49) ~[eventmesh-runtime-1.10.0-release.jar:1.10.0-release]
at java.util.HashMap.computeIfAbsent(HashMap.java:1128) ~[?:1.8.0_342]
at org.apache.eventmesh.runtime.trace.Trace.getInstance(Trace.java:49) ~[eventmesh-runtime-1.10.0-release.jar:1.10.0-release]
at org.apache.eventmesh.runtime.boot.EventMeshServer.(EventMeshServer.java:105) ~[eventmesh-runtime-1.10.0-release.jar:1.10.0-release]
at org.apache.eventmesh.runtime.boot.EventMeshStartup.main(EventMeshStartup.java:37) [eventmesh-runtime-1.10.0-release.jar:1.10.0-release]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Startup failed due to NullPointerException.

4 participants