Skip to content

Commit c084198

Browse files
authored
Merge pull request #14 from nubank/DGD-4411-java8-compatible-build
Altering NuEventEmitter to be compatible with jdk 8
2 parents 8a8e2f6 + c9f4806 commit c084198

File tree

1 file changed

+13
-6
lines changed

1 file changed

+13
-6
lines changed

integration/spark/app/src/main/java/io/openlineage/spark/agent/NuEventEmitter.java

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@
44
import lombok.extern.slf4j.Slf4j;
55

66
import java.lang.reflect.Field;
7+
import java.util.Arrays;
8+
import java.util.Collections;
9+
import java.util.HashSet;
710
import java.util.Set;
811
import java.util.stream.Collectors;
912
import java.util.stream.Stream;
@@ -16,13 +19,17 @@
1619
@Slf4j
1720
public class NuEventEmitter {
1821

19-
private static final Set<String> WANTED_JOB_TYPES = Set.of(
20-
"SQL_JOB" // as defined in SparkSQLExecutionContext.SPARK_JOB_TYPE
22+
private static final Set<String> WANTED_JOB_TYPES = new HashSet<>(
23+
Collections.singletonList(
24+
"SQL_JOB" // as defined in SparkSQLExecutionContext.SPARK_JOB_TYPE
25+
)
2126
);
2227

23-
private static final Set<String> WANTED_EVENT_NAME_SUBSTRINGS = Set.of(
24-
".execute_insert_into_hadoop_fs_relation_command.",
25-
".adaptive_spark_plan."
28+
private static final Set<String> WANTED_EVENT_NAME_SUBSTRINGS = new HashSet<>(
29+
Arrays.asList(
30+
".execute_insert_into_hadoop_fs_relation_command.",
31+
".adaptive_spark_plan."
32+
)
2633
);
2734

2835
private static Boolean isPermittedJobType(RunEvent event) {
@@ -49,7 +56,7 @@ private static Boolean isPermittedJobName(RunEvent event) {
4956
return false;
5057
}
5158
if (WANTED_EVENT_NAME_SUBSTRINGS.stream().noneMatch(jobName::contains)) {
52-
log.debug("OpenLineage event job name has no permitted substring and should not be emitted");
59+
log.debug("OpenLineage event job name {} has no permitted substring and should not be emitted", jobName);
5360
return false;
5461
}
5562
return true;

0 commit comments

Comments
 (0)