Skip to content
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

Map should fill null values only when set WriteMapNullValue #1837

Closed
wants to merge 2,693 commits into from

Conversation

jasonjoo2010
Copy link

make it more compatible to previous versions.

wenshao and others added 30 commits September 24, 2017 18:46
修复在FastJsonHttpMessageConverter 中使用了spring4.x版本新增的TypeResolvable 导致sp…
wenshao and others added 24 commits February 6, 2018 16:20
add MappingFastJsonMessageConverter and testcase for alibaba#1736
bug fixed for issue alibaba#1763,如果有多层泛型且前面泛型已经实现的情况下,判断下一级泛型
@CLAassistant
Copy link

CLAassistant commented May 8, 2018

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov-io
Copy link

Codecov Report

Merging #1837 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1837      +/-   ##
============================================
- Coverage     81.06%   81.05%   -0.01%     
+ Complexity     6210     6209       -1     
============================================
  Files           137      137              
  Lines         23081    23081              
  Branches       5343     5343              
============================================
- Hits          18711    18709       -2     
- Misses         3024     3025       +1     
- Partials       1346     1347       +1
Impacted Files Coverage Δ Complexity Δ
...m/alibaba/fastjson/serializer/FieldSerializer.java 85.71% <100%> (ø) 57 <0> (ø) ⬇️
...alibaba/fastjson/serializer/SerializerFeature.java 92.72% <100%> (ø) 9 <0> (ø) ⬇️
...baba/fastjson/serializer/ASMSerializerFactory.java 97.7% <100%> (ø) 211 <0> (ø) ⬇️
...com/alibaba/fastjson/serializer/MapSerializer.java 75% <100%> (ø) 48 <0> (ø) ⬇️
...com/alibaba/fastjson/parser/JSONReaderScanner.java 72.61% <0%> (-1.28%) 44% <0%> (-1%)

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 af36516...902dd79. Read the comment docs.

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.