You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2022/06/14 07:55:22 UTC
[flink] 01/02: [hotfix] Cleanup EnumValueSerializerCompatibilityTest
This is an automated email from the ASF dual-hosted git repository.
chesnay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git
commit cc59fab54ac7ae904413c18d41f24abca0f7501d
Author: Chesnay Schepler <ch...@apache.org>
AuthorDate: Wed Dec 1 12:11:27 2021 +0100
[hotfix] Cleanup EnumValueSerializerCompatibilityTest
- 'in' variable was unused
- 'out' variable was unnecessary because the reporter by default prints to stderr anyway, and there isn't a real benefit in using stdout
---
.../api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/flink-scala/src/test/scala/org/apache/flink/api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala b/flink-scala/src/test/scala/org/apache/flink/api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala
index a5dcc619b5a..b077933c5f7 100644
--- a/flink-scala/src/test/scala/org/apache/flink/api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala
+++ b/flink-scala/src/test/scala/org/apache/flink/api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala
@@ -30,7 +30,7 @@ import java.io._
import java.net.{URL, URLClassLoader}
import scala.reflect.NameTransformer
-import scala.tools.nsc.{GenericRunnerSettings, Global}
+import scala.tools.nsc.{Global, Settings}
import scala.tools.nsc.reporters.ConsoleReporter
class EnumValueSerializerCompatibilityTest extends TestLogger with JUnitSuiteLike {
@@ -177,10 +177,7 @@ object EnumValueSerializerCompatibilityTest {
}
def compileScalaFile(file: File): Unit = {
- val in = new BufferedReader(new StringReader(""))
- val out = new PrintWriter(new BufferedWriter(new OutputStreamWriter(System.out)))
-
- val settings = new GenericRunnerSettings(out.println _)
+ val settings = new Settings()
// use the java classpath so that scala libraries are available to the compiler
settings.usejavacp.value = true