Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1045#discussion_r162250517
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/store/TestAffinityCalculator.java
---
@@ -21,18 +21,14 @@
import org.apache.drill.exec.ExecTest;
import org.apache.drill.exec.proto.CoordinationProtos;
-import org.apache.drill.exec.store.parquet.ParquetGroupScan;
import org.apache.hadoop.fs.BlockLocation;
import org.junit.Test;
import com.google.common.collect.ImmutableRangeMap;
import com.google.common.collect.Range;
public class TestAffinityCalculator extends ExecTest {
- static final org.slf4j.Logger logger =
org.slf4j.LoggerFactory.getLogger(TestAffinityCalculator.class);
-
- String port = "1234";
- final String path = "path";
+ private final String port = "1234";
--- End diff --
What happened to the blocks of code that were removed? Not used? Duplicate?
Or, does the unused code suggest that this test is not actually testing what it
should?
---
