Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/1251#discussion_r230517250
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/BaseFunctionResolver.java
---
@@ -94,6 +95,16 @@ public void initialize(Context context) {
this.context = context;
}
+ /**
+ * Close the Stellar functions.
+ */
+ @Override
+ public void close() throws IOException {
+ for (StellarFunctionInfo info : getFunctionInfo()) {
--- End diff --
Should there be an "Attempt" to close each function? I think we want to
have a try catch for each call here maybe and just throw with the functions
that failed afterwards.
---
