Skip to content

remove useless info#2673

Closed
IanCao wants to merge 29 commits intoapache:masterfrom
IanCao:master
Closed

remove useless info#2673
IanCao wants to merge 29 commits intoapache:masterfrom
IanCao:master

Conversation

@IanCao
Copy link
Copy Markdown
Contributor

@IanCao IanCao commented May 15, 2019

This oversize spans Exeception is useless. but this will make more log info.
so i suggest to remove this

@IanCao IanCao requested a review from wu-sheng May 15, 2019 06:03
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.007%) to 16.491% when pulling a37ec5f on IanCao:master into 91ffae2 on apache:master.

Copy link
Copy Markdown
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

This exception is intended to do, make log component shows the stack when this happens.

@wu-sheng wu-sheng closed this May 15, 2019
@wu-sheng wu-sheng added the wontfix This will not be worked on label May 15, 2019
@wu-sheng wu-sheng added this to the 6.2.0 milestone May 15, 2019
@IanCao
Copy link
Copy Markdown
Contributor Author

IanCao commented May 15, 2019

make sense

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

Labels

wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants