Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EventDrivenUpdateGraph. #4613

Merged
merged 33 commits into from
Dec 5, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
5ae47e6
EventDrivenUpdateGraph.
cpwright Oct 10, 2023
9e51eb5
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Oct 12, 2023
fd2f6ab
bad import
cpwright Oct 12, 2023
36cf076
Comment on PoisonedNotificationProcessor.
cpwright Oct 12, 2023
16ffa3b
running into state problems.
cpwright Oct 12, 2023
b77d836
rename stuff
cpwright Oct 16, 2023
5a5e078
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Oct 17, 2023
bc7df8a
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Oct 19, 2023
662280c
move graph name collision check into base.
cpwright Oct 19, 2023
7dcb72f
test, but in an impossible place
cpwright Oct 20, 2023
823ce85
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Oct 23, 2023
5a42f31
Use EDUG for default.
cpwright Oct 24, 2023
7a9fa30
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Oct 24, 2023
bfac9cc
flush reset
cpwright Oct 24, 2023
4fc1e2b
also clear on end
cpwright Oct 24, 2023
9b4b295
some comments
cpwright Oct 24, 2023
990b92f
spotless
cpwright Oct 24, 2023
434b338
self-review.
cpwright Oct 24, 2023
0bfac33
rset UPT.
cpwright Oct 24, 2023
0c4790c
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Oct 26, 2023
3977fd8
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Oct 26, 2023
19cfb88
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Oct 27, 2023
c66f342
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Nov 29, 2023
678af75
Merge remote-tracking branch 'upstream/main' into cpw_edug
cpwright Dec 5, 2023
c182b5d
live code review comments to address.
cpwright Dec 5, 2023
f7e6dca
cleanup
cpwright Dec 5, 2023
d326db9
spotless
cpwright Dec 5, 2023
793f744
javadoc
cpwright Dec 5, 2023
ee0fa06
Cleanup of unused variables, move budget calculation.
cpwright Dec 5, 2023
0e3add2
Fix existingOrBuild.
cpwright Dec 5, 2023
dcc53f8
speling.
cpwright Dec 5, 2023
7eaa896
Apply suggestions from code review
cpwright Dec 5, 2023
407bda9
spotless
cpwright Dec 5, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -118,4 +118,7 @@ public boolean supportsRefreshing() {
public void requestRefresh() {
fail();
}

@Override
public void stop() {}
}
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,14 @@
import io.deephaven.engine.tablelogger.EngineTableLoggers;
import io.deephaven.engine.tablelogger.UpdatePerformanceLogLogger;
import io.deephaven.engine.updategraph.UpdateGraph;
import io.deephaven.engine.updategraph.impl.PeriodicUpdateGraph;
import io.deephaven.engine.updategraph.impl.BaseUpdateGraph;
import io.deephaven.engine.util.string.StringUtils;
import io.deephaven.internal.log.LoggerFactory;
import io.deephaven.io.logger.Logger;
import io.deephaven.stream.StreamToBlinkTableAdapter;
import io.deephaven.util.QueryConstants;
import io.deephaven.util.SafeCloseable;
import io.deephaven.util.annotations.TestUseOnly;
import org.apache.commons.lang3.mutable.MutableObject;
import org.jetbrains.annotations.NotNull;

Expand All @@ -37,7 +38,7 @@

/**
* <p>
* This tool is meant to track periodic update events that take place in an {@link PeriodicUpdateGraph}. This generally
* This tool is meant to track periodic update events that take place in an {@link UpdateGraph}. This generally
* includes:
* <ol>
* <li>Update source {@code run()} invocations</li>
Expand Down Expand Up @@ -88,8 +89,8 @@ private static class InternalState {

private InternalState() {
final UpdateGraph publishingGraph =
PeriodicUpdateGraph.getInstance(PeriodicUpdateGraph.DEFAULT_UPDATE_GRAPH_NAME);
Assert.neqNull(publishingGraph, "The " + PeriodicUpdateGraph.DEFAULT_UPDATE_GRAPH_NAME + " UpdateGraph "
BaseUpdateGraph.getInstance(BaseUpdateGraph.DEFAULT_UPDATE_GRAPH_NAME);
Assert.neqNull(publishingGraph, "The " + BaseUpdateGraph.DEFAULT_UPDATE_GRAPH_NAME + " UpdateGraph "
+ "must be created before UpdatePerformanceTracker can be initialized.");
try (final SafeCloseable ignored = ExecutionContext.getContext().withUpdateGraph(publishingGraph).open()) {
tableLogger = EngineTableLoggers.get().updatePerformanceLogLogger();
Expand Down Expand Up @@ -286,4 +287,11 @@ public long getIntervalEndTimeEpochNanos() {
public static QueryTable getQueryTable() {
return (QueryTable) BlinkTableTools.blinkToAppendOnly(getInternalState().blink);
}

@TestUseOnly
public static void resetForUnitTests() {
synchronized (UpdatePerformanceTracker.class) {
INSTANCE = null;
}
}
}
Loading
Loading