Github user jeking3 commented on a diff in the pull request:
https://github.com/apache/thrift/pull/1496#discussion_r174494320
--- Diff: lib/java/gradle/unitTests.gradle ---
@@ -68,6 +68,20 @@ test {
include '**/Test*.class'
exclude '**/Test*\$*.class'
+ println 'Check if msvc.profile exists'
--- End diff --
It seems to me this fix is in the wrong place. I don't like having the C++
compiler type passed down into the java build so it can disable a test. The
compiler code be modified somehow.---
