fix memory leaks

Author: DimitrisStaratzis

build.gradle

@@ -9,7 +9,7 @@ plugins {
 }
 
 group 'io.tiledb'
-version '0.19.0-SNAPSHOT'
+version '0.19.82'
 
 repositories {
     jcenter()

src/main/java/io/tiledb/java/api/Array.java

@@ -963,14 +963,14 @@ protected SWIGTYPE_p_tiledb_array_t getArrayp() {
 
   /** Free's the native objects and closes the Array. */
   public synchronized void close() {
+    if (schema != null) {
+      schema.close();
+    }
     if (arrayp != null && arraypp != null) {
       tiledb.tiledb_array_close(ctx.getCtxp(), arrayp);
       tiledb.tiledb_array_free(arraypp);
       arrayp = null;
       arraypp = null;
-      if (schema != null) {
-        schema.close();
-      }
     }
   }
 }

src/main/java/io/tiledb/java/api/ArraySchema.java

@@ -802,15 +802,9 @@ public String toString() {
   public void close() {
     if (schemap != null) {
       tiledb.tiledb_array_schema_free(schemapp);
+      tiledb.delete_tiledb_array_schema_tpp(schemapp);
       schemap = null;
       schemapp = null;
     }
-    // TODO: remove
-    if (attributes != null) {
-      for (Map.Entry<String, Attribute> e : attributes.entrySet()) {
-        e.getValue().close();
-      }
-    }
-    attributes = null;
   }
 }

src/main/java/io/tiledb/java/api/ArraySchemaEvolution.java

@@ -41,6 +41,7 @@ protected Context getCtx() {
   public void close() {
     if (evolutionp != null && evolutionpp != null) {
       tiledb.tiledb_array_schema_evolution_free(evolutionpp);
+      tiledb.delete_tiledb_array_schema_evolution_tpp(evolutionpp);
       evolutionpp = null;
       evolutionp = null;
     }

src/main/java/io/tiledb/java/api/Attribute.java

@@ -546,6 +546,7 @@ public String toString() {
   public void close() {
     if (attributep != null) {
       tiledb.tiledb_attribute_free(attributepp);
+      tiledb.delete_tiledb_attribute_tpp(attributepp);
       attributep = null;
       attributepp = null;
     }

src/main/java/io/tiledb/java/api/Config.java

@@ -357,6 +357,7 @@ protected SWIGTYPE_p_tiledb_config_t getConfigp() {
   public void close() {
     if (configp != null) {
       tiledb.tiledb_config_free(configpp);
+      tiledb.delete_tiledb_config_tpp(configpp);
       configp = null;
       configpp = null;
     }

src/main/java/io/tiledb/java/api/ConsolidationPlan.java

@@ -132,6 +132,7 @@ public long getNumNodes() throws TileDBError {
   public void close() throws Exception {
     if (consp != null) {
       tiledb.tiledb_consolidation_plan_free(conspp);
+      tiledb.delete_tiledb_consolidation_plan_tpp(conspp);
       consp = null;
       conspp = null;
     }

src/main/java/io/tiledb/java/api/Context.java

@@ -261,6 +261,7 @@ public String getStats() throws TileDBError {
   public void close() {
     if (ctxp != null) {
       tiledb.tiledb_ctx_free(ctxpp);
+      tiledb.delete_tiledb_ctx_tpp(ctxpp);
       ctxp = null;
       ctxpp = null;
     }

src/main/java/io/tiledb/java/api/Dimension.java

@@ -310,6 +310,7 @@ public String tileExtentToStr() throws TileDBError {
   public void close() {
     if (dimensionp != null) {
       tiledb.tiledb_dimension_free(dimensionpp);
+      tiledb.delete_tiledb_dimension_tpp(dimensionpp);
       dimensionp = null;
       dimensionpp = null;
     }

src/main/java/io/tiledb/java/api/Domain.java

@@ -289,11 +289,7 @@ public void addDimensions(Collection<Dimension> dims) throws TileDBError {
   public void close() {
     if (domainp != null) {
       tiledb.tiledb_domain_free(domainpp);
-    }
-    if (dimensions != null) {
-      for (Dimension d : dimensions) {
-        d.close();
-      }
+      tiledb.delete_tiledb_domain_tpp(domainpp);
     }
   }
 }