Skip to content

Commit

Permalink
Update Integrations/src/main/java/io/deephaven/integrations/python/Py…
Browse files Browse the repository at this point in the history
…thonMergedListenerAdapter.java

Co-authored-by: Ryan Caudy <[email protected]>
  • Loading branch information
jmao-denver and rcaudy committed Jul 10, 2024
1 parent 6abc8ac commit d08a36d
Showing 1 changed file with 0 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -54,15 +54,6 @@ public static PythonMergedListenerAdapter create(
if (recorders.length < 2) {
throw new IllegalArgumentException("At least two recorders must be provided");
}
// TODO: Uncomment this check if confirmed that the alternative way of checking is better
// final UpdateGraph updateGraph = ExecutionContext.getContext().getUpdateGraph();
// if (!Arrays.stream(recorders).allMatch(t -> t.getParent().getUpdateGraph() == updateGraph)) {
// throw new IllegalArgumentException("All recorders must be from the same update graph");
// }
//
// if (!Arrays.stream(dependencies).allMatch(t -> t.getUpdateGraph() == updateGraph)) {
// throw new IllegalArgumentException("All dependencies must be from the same update graph");
// }

final NotificationQueue.Dependency[] allItems =
Stream.concat(Arrays.stream(recorders), Arrays.stream(dependencies))
Expand Down

0 comments on commit d08a36d

Please sign in to comment.