DimensionLookup transform fails with hop-run apache#4328

Adalennis · Oct 1, 2024 · cc8f8cf · cc8f8cf
1 parent 5699cb6
commit cc8f8cf
Showing 1 changed file with 20 additions and 32 deletions.
diff --git a/...src/main/java/org/apache/hop/pipeline/transforms/dimensionlookup/DimensionLookupMeta.java b/...src/main/java/org/apache/hop/pipeline/transforms/dimensionlookup/DimensionLookupMeta.java
@@ -281,44 +281,32 @@ public void getFields(
     }
 
     try {
-      DatabaseMeta databaseMeta =
-          metadataProvider.getSerializer(DatabaseMeta.class).load(variables.resolve(connection));
-
-      try (Database db = new Database(loggingObject, variables, databaseMeta)) {
-        // Get the rows from the table...
-        IRowMeta extraFields = db.getTableFieldsMeta(schemaName, tableName);
+      // Get the rows from the table...
+      IRowMeta extraFields = getTableFields(variables);
 
-        for (DLField field : fields.fields) {
-          v = extraFields.searchValueMeta(field.getLookup());
-          if (v == null) {
-            String message =
-                BaseMessages.getString(
-                    PKG,
-                    "DimensionLookupMeta.Exception.UnableToFindReturnField",
-                    field.getLookup());
-            logError(message);
-            throw new HopTransformException(message);
-          }
+      for (DLField field : fields.fields) {
+        v = extraFields.searchValueMeta(field.getLookup());
+        if (v == null) {
+          String message =
+              BaseMessages.getString(
+                  PKG, "DimensionLookupMeta.Exception.UnableToFindReturnField", field.getLookup());
+          logError(message);
+          throw new HopTransformException(message);
+        }
 
-          // If the field needs to be renamed, rename
-          if (StringUtils.isNotEmpty(field.getName())) {
-            v.setName(field.getName());
-          }
-          v.setOrigin(name);
-          row.addValueMeta(v);
+        // If the field needs to be renamed, rename
+        if (StringUtils.isNotEmpty(field.getName())) {
+          v.setName(field.getName());
         }
-      } catch (Exception e) {
-        String message =
-            BaseMessages.getString(
-                PKG, "DimensionLookupMeta.Exception.UnableToRetrieveDataTypeOfReturnField2");
-        logError(message);
-        throw new HopTransformException(message, e);
+        v.setOrigin(name);
+        row.addValueMeta(v);
       }
-    } catch (HopException e) {
+    } catch (Exception e) {
       String message =
-          BaseMessages.getString(PKG, "DimensionLookupMeta.Exception.DatabaseErrorOccurred")
-              + e.getMessage();
+          BaseMessages.getString(
+              PKG, "DimensionLookupMeta.Exception.UnableToRetrieveDataTypeOfReturnField2");
       logError(message);
+      throw new HopTransformException(message, e);
     }
   }