Skip to content

Add missing property set for new field httpResponseStatusCode#7486

Merged
wu-sheng merged 2 commits into
masterfrom
bugfix/missing-httpResponseStatusCode
Aug 18, 2021
Merged

Add missing property set for new field httpResponseStatusCode#7486
wu-sheng merged 2 commits into
masterfrom
bugfix/missing-httpResponseStatusCode

Conversation

@kezhenxu94
Copy link
Copy Markdown
Member

@kezhenxu94 kezhenxu94 added bug Something isn't working and you are sure it's a bug! backend OAP backend related. labels Aug 18, 2021
@kezhenxu94 kezhenxu94 added this to the 8.8.0 milestone Aug 18, 2021
@kezhenxu94 kezhenxu94 requested a review from wu-sheng August 18, 2021 15:34
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 18, 2021

Codecov Report

Merging #7486 (a5d4124) into master (39d23c1) will decrease coverage by 9.83%.
The diff coverage is 76.56%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #7486      +/-   ##
============================================
- Coverage     58.81%   48.98%   -9.84%     
+ Complexity     4390     3874     -516     
============================================
  Files          1035     1901     +866     
  Lines         26592    41258   +14666     
  Branches       2627     4619    +1992     
============================================
+ Hits          15640    20210    +4570     
- Misses         9571    19930   +10359     
+ Partials       1381     1118     -263     
Impacted Files Coverage Δ
.../provider/log/listener/RecordAnalysisListener.java 72.41% <ø> (ø)
...ng/oap/server/core/query/type/ServiceInstance.java 0.00% <ø> (ø)
...alking/oap/query/graphql/GraphQLQueryProvider.java 0.00% <0.00%> (ø)
.../server/receiver/mesh/TelemetryDataDispatcher.java 7.00% <0.00%> (-77.77%) ⬇️
...g/oap/log/analyzer/dsl/spec/filter/FilterSpec.java 65.33% <30.76%> (+23.79%) ⬆️
...pache/skywalking/oap/log/analyzer/dsl/Binding.java 77.55% <50.00%> (+16.57%) ⬆️
...log/analyzer/dsl/spec/extractor/ExtractorSpec.java 50.00% <75.00%> (+23.00%) ⬆️
...lking/oap/query/graphql/resolver/LogTestQuery.java 88.13% <88.13%> (ø)
...rg/apache/skywalking/oap/log/analyzer/dsl/DSL.java 100.00% <100.00%> (ø)
.../apache/skywalking/oap/meter/analyzer/dsl/DSL.java 96.29% <100.00%> (+6.29%) ⬆️
... and 1385 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c04b4e0...a5d4124. Read the comment docs.

@wu-sheng wu-sheng merged commit ad0042f into master Aug 18, 2021
@wu-sheng wu-sheng deleted the bugfix/missing-httpResponseStatusCode branch August 18, 2021 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend OAP backend related. bug Something isn't working and you are sure it's a bug!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants