----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65761/#review198151 -----------------------------------------------------------
Master (d95fc2f) is green with this patch. ./build-support/jenkins/build.sh However, it appears that it might lack test coverage. I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Feb. 22, 2018, 11:23 p.m., Jordan Ly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65761/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2018, 11:23 p.m.) > > > Review request for Aurora and Renan DelValle. > > > Repository: aurora > > > Description > ------- > > Minor nit, use the StandardCharset constant for UTF-8 as opposed to creating > it ourselves. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/http/api/ApiModule.java > 09950a77427c7ed94090a036ae26dae2939aa2c1 > > > Diff: https://reviews.apache.org/r/65761/diff/1/ > > > Testing > ------- > > `./gradlew test` > > Manually verified Firefox, Chrome, and Safari. > > > Thanks, > > Jordan Ly > >