Skip to content

Conversation

@wujimin
Copy link
Contributor

@wujimin wujimin commented Nov 29, 2017

it's netty's bug.
latest vertx had upgrade netty dependence.
we can upgrade to latest vertx 3.5.0

@coveralls
Copy link

Coverage Status

Coverage increased (+0.002%) to 86.986% when pulling 740b589 on wujimin:fix-dns-resolve into 7ddf345 on ServiceComb:master.

@@ -0,0 +1,69 @@
package io.vertx.ext.web.impl;
Copy link
Member

Choose a reason for hiding this comment

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

Please add license header.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -0,0 +1,81 @@
package io.vertx.ext.web.impl;
Copy link
Member

Choose a reason for hiding this comment

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

License header.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


<properties>
<jackson.version>2.8.10</jackson.version>
<vertx.version>3.5.0</vertx.version>
Copy link
Member

Choose a reason for hiding this comment

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

This file is reformated, Can we just update the version ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

first commit in yellow cloud, forgot if reformat it.
it's not important, right?

@coveralls
Copy link

Coverage Status

Coverage increased (+0.002%) to 86.973% when pulling a4a8987 on wujimin:fix-dns-resolve into 264bb14 on ServiceComb:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 86.945% when pulling 738db83 on wujimin:fix-dns-resolve into 856657d on ServiceComb:master.

@liubao68 liubao68 merged commit 64e6449 into apache:master Dec 2, 2017
@wujimin wujimin deleted the fix-dns-resolve branch January 1, 2018 09:02
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.

4 participants