You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by cu...@apache.org on 2019/01/24 01:56:42 UTC
[arrow] branch master updated: ARROW-4344: [Java] Further cleanup
mvn output, upgrade rat plugin
This is an automated email from the ASF dual-hosted git repository.
cutlerb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/master by this push:
new 28a965e ARROW-4344: [Java] Further cleanup mvn output, upgrade rat plugin
28a965e is described below
commit 28a965e2066f5143689142e4d903e8c8af424767
Author: Bryan Cutler <cu...@gmail.com>
AuthorDate: Wed Jan 23 17:56:29 2019 -0800
ARROW-4344: [Java] Further cleanup mvn output, upgrade rat plugin
Further cleanup of mvn output to help reduce CI log clutter. Changed EchoServer used in tests to have log level of debug. Upgraded rat license check plugin to 0.13. The previous version made lots of output just showing what file types were being excluded. The upgraded version omits this, but it can be enabled by setting --debug.
Author: Bryan Cutler <cu...@gmail.com>
Closes #3466 from BryanCutler/java-further-mvn-cleanup-ARROW-4344 and squashes the following commits:
65e0dce <Bryan Cutler> Change EchoServer logging level to debug for tests
b58cd03 <Bryan Cutler> Upgrade Java rat plugin to 0.13
---
java/pom.xml | 2 +-
.../src/main/java/org/apache/arrow/tools/EchoServer.java | 14 +++++++-------
2 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/java/pom.xml b/java/pom.xml
index 96385c0..95b8bd5 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -350,7 +350,7 @@
<plugin>
<groupId>org.apache.rat</groupId>
<artifactId>apache-rat-plugin</artifactId>
- <version>0.11</version>
+ <version>0.13</version>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
diff --git a/java/tools/src/main/java/org/apache/arrow/tools/EchoServer.java b/java/tools/src/main/java/org/apache/arrow/tools/EchoServer.java
index 95ff710..6f68a9e 100644
--- a/java/tools/src/main/java/org/apache/arrow/tools/EchoServer.java
+++ b/java/tools/src/main/java/org/apache/arrow/tools/EchoServer.java
@@ -37,9 +37,9 @@ public class EchoServer {
private boolean closed = false;
public EchoServer(int port) throws IOException {
- LOGGER.info("Starting echo server.");
+ LOGGER.debug("Starting echo server.");
serverSocket = new ServerSocket(port);
- LOGGER.info("Running echo server on port: " + port());
+ LOGGER.debug("Running echo server on port: " + port());
}
public static void main(String[] args) throws Exception {
@@ -59,9 +59,9 @@ public class EchoServer {
public void run() throws IOException {
try {
while (!closed) {
- LOGGER.info("Waiting to accept new client connection.");
+ LOGGER.debug("Waiting to accept new client connection.");
Socket clientSocket = serverSocket.accept();
- LOGGER.info("Accepted new client connection.");
+ LOGGER.debug("Accepted new client connection.");
try (ClientConnection client = new ClientConnection(clientSocket)) {
try {
client.run();
@@ -69,7 +69,7 @@ public class EchoServer {
LOGGER.warn("Error handling client connection.", e);
}
}
- LOGGER.info("Closed connection with client");
+ LOGGER.debug("Closed connection with client");
}
} catch (java.net.SocketException ex) {
if (!closed) {
@@ -77,7 +77,7 @@ public class EchoServer {
}
} finally {
serverSocket.close();
- LOGGER.info("Server closed.");
+ LOGGER.debug("Server closed.");
}
}
@@ -116,7 +116,7 @@ public class EchoServer {
}
writer.end();
Preconditions.checkState(reader.bytesRead() == writer.bytesWritten());
- LOGGER.info(String.format("Echoed %d records", echoed));
+ LOGGER.debug(String.format("Echoed %d records", echoed));
}
}
}