Skip to content

KYLIN-4254: The result exporting from Insight with CSV format is empty, when sql contains Chinese#938

Closed
zhangxiang17 wants to merge 2 commits intoapache:masterfrom
zhangxiang17:bugfix
Closed

KYLIN-4254: The result exporting from Insight with CSV format is empty, when sql contains Chinese#938
zhangxiang17 wants to merge 2 commits intoapache:masterfrom
zhangxiang17:bugfix

Conversation

@zhangxiang17
Copy link

  1. change the tomcat's default decoding to 'UTF-8' in server.xml

@asf-ci
Copy link

asf-ci commented Nov 13, 2019

Can one of the admins verify this patch?

@zhangxiang17 zhangxiang17 changed the title The result exporting from Insight with CSV format is empty, when sql contains Chinese KYLIN-4254: The result exporting from Insight with CSV format is empty, when sql contains Chinese Nov 13, 2019
@codecov-io
Copy link

Codecov Report

Merging #938 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #938   +/-   ##
=========================================
  Coverage     25.57%   25.57%           
  Complexity     6139     6139           
=========================================
  Files          1412     1412           
  Lines         85072    85072           
  Branches      11928    11928           
=========================================
  Hits          21757    21757           
  Misses        61208    61208           
  Partials       2107     2107
Impacted Files Coverage Δ Complexity Δ
...n/rest/security/PasswordPlaceholderConfigurer.java 0% <ø> (ø) 0 <0> (ø) ⬇️

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 7de98cf...885a576. Read the comment docs.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 5217

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 28.055%

Totals Coverage Status
Change from base Build 5214: 0.0%
Covered Lines: 23867
Relevant Lines: 85072

💛 - Coveralls

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