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

feature:gzip compressor #1460

Merged
merged 23 commits into from
Oct 22, 2019
Merged

feature:gzip compressor #1460

merged 23 commits into from
Oct 22, 2019

Conversation

jsbxyyx
Copy link
Member

@jsbxyyx jsbxyyx commented Aug 14, 2019

Ⅰ. Describe what this PR did

feature gzip compressor

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov-io
Copy link

codecov-io commented Aug 14, 2019

Codecov Report

Merging #1460 into develop will decrease coverage by <.01%.
The diff coverage is 52.72%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #1460      +/-   ##
=============================================
- Coverage      55.06%   55.06%   -0.01%     
- Complexity      2345     2353       +8     
=============================================
  Files            418      421       +3     
  Lines          14197    14250      +53     
  Branches        1701     1705       +4     
=============================================
+ Hits            7818     7847      +29     
- Misses          5658     5680      +22     
- Partials         721      723       +2
Impacted Files Coverage Δ Complexity Δ
.../java/io/seata/core/compressor/CompressorType.java 0% <ø> (ø) 0 <0> (?)
...java/io/seata/core/protocol/ProtocolConstants.java 0% <ø> (ø) 0 <0> (ø) ⬇️
...va/io/seata/core/compressor/CompressorFactory.java 0% <0%> (ø) 0 <0> (?)
.../io/seata/core/rpc/netty/v1/ProtocolV1Decoder.java 0% <0%> (ø) 0 <0> (ø) ⬇️
.../io/seata/core/rpc/netty/v1/ProtocolV1Encoder.java 0% <0%> (ø) 0 <0> (ø) ⬇️
.../java/io/seata/compressor/gzip/GzipCompressor.java 100% <100%> (ø) 3 <3> (?)
...c/main/java/io/seata/compressor/gzip/GzipUtil.java 78.78% <78.78%> (ø) 5 <5> (?)

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 7e1bc6b...c4710fd. Read the comment docs.

@xingfudeshi
Copy link
Member

@jsbxyyx CI is failed.

@jsbxyyx
Copy link
Member Author

jsbxyyx commented Sep 8, 2019

@xingfudeshi the issue 1564 cause CI failed.

@l81893521
Copy link
Contributor

I think if you can provide the unit test for CompressorFactory, that s better

Copy link
Contributor

@l81893521 l81893521 left a comment

Choose a reason for hiding this comment

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

should add dependency to server/pom.xml

Copy link
Contributor

@l81893521 l81893521 left a comment

Choose a reason for hiding this comment

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

Test passed. LGTM

Copy link
Member

@xingfudeshi xingfudeshi left a comment

Choose a reason for hiding this comment

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

LGTM.

@xingfudeshi xingfudeshi changed the title feature gzip compressor feature:gzip compressor Sep 30, 2019
@slievrly slievrly added this to the 1.0 milestone Oct 10, 2019
@zjinlei zjinlei added the Do Not Merge Do not merge into develop label Oct 22, 2019
@zjinlei zjinlei merged commit 0cad72e into apache:develop Oct 22, 2019
@jsbxyyx jsbxyyx deleted the zip branch October 22, 2019 09:13
dangquocbang pushed a commit to dangquocbang/seata that referenced this pull request Oct 27, 2019
* 'develop' of https://github.com/seata/seata:
  Remove deprecated JVM arguments in Java 11 (apache#1824)
  style: adjust check style (apache#1820)
  optimize: format error log (apache#1806)
  optimize: update codecov.yml (apache#1815)
  optimize: adjust codecov configuration (apache#1811)
  fix unexcepted exception can roll back (apache#1796)
  optimize: reduce unnecessary synchronized (apache#1799)
  bugfix: connectionproxy prepareStatement not in global transaction (apache#1805)
  bugfix: fix can't use select for update in oracle (apache#1780)
  test:changing HashMap to LinkedHashMap for deterministic iterations (apache#1802)
  feature:gzip compressor (apache#1460)
@zjinlei zjinlei removed the Do Not Merge Do not merge into develop label Nov 16, 2019
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.

6 participants