Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1142#discussion_r171606555
  
    --- Diff: 
contrib/storage-opentsdb/src/test/java/org/apache/drill/store/openTSDB/TestOpenTSDBPlugin.java
 ---
    @@ -51,17 +54,17 @@
     
     public class TestOpenTSDBPlugin extends PlanTestBase {
     
    -  protected static OpenTSDBStoragePlugin storagePlugin;
    -  protected static OpenTSDBStoragePluginConfig storagePluginConfig;
    +  private static int portNum = 10_000;
    --- End diff --
    
    Why do you set value right away? It looks you will always re-write in 
`setup`.


---

Reply via email to