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

Update Clock interface, remove TimeProvider #2998

Merged
merged 24 commits into from
Oct 28, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
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
2 changes: 2 additions & 0 deletions Base/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ plugins {

dependencies {

api project(':clock')

implementation depTrove3
compileOnlyApi depAnnotations
implementation depCommonsLang3
Expand Down
89 changes: 0 additions & 89 deletions Base/src/main/java/io/deephaven/base/clock/Clock.java

This file was deleted.

1 change: 0 additions & 1 deletion Configuration/src/main/resources/lib-tests.prop
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,4 @@ NIO.driver.maxThreadCount=400
NIO.driver.useFast=true

Comm.fatalErrorHandlerFactoryClass=io.deephaven.base.ThrowRuntimeExceptionFatalErrorHandlerFactory
NIO.wireLagClock.native=false
default.processEnvironmentFactory=io.deephaven.util.process.DefaultProcessEnvironment$Factory
1 change: 0 additions & 1 deletion Configuration/src/test/resources/resources/lib-tests.prop
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,4 @@ NIO.driver.maxThreadCount=400
NIO.driver.useFast=true

Comm.fatalErrorHandlerFactoryClass=io.deephaven.base.ThrowRuntimeExceptionFatalErrorHandlerFactory
NIO.wireLagClock.native=false
default.processEnvironmentFactory=io.deephaven.util.process.DefaultProcessEnvironment$Factory

This file was deleted.

21 changes: 0 additions & 21 deletions FishUtil/src/main/java/io/deephaven/util/clock/LocalClockImpl.java

This file was deleted.

104 changes: 0 additions & 104 deletions FishUtil/src/main/java/io/deephaven/util/clock/MicroTimer.java

This file was deleted.

22 changes: 0 additions & 22 deletions FishUtil/src/main/java/io/deephaven/util/clock/RealTimeClock.java

This file was deleted.

53 changes: 0 additions & 53 deletions Util/src/main/java/io/deephaven/util/ClockFactory.java

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import org.gradle.api.Project
class CombinedJavadoc {

static boolean includeProject(Project p) {
def exclusions = [ 'hotspot-impl' ].toSet()
def exclusions = [ 'hotspot-impl', 'clock-impl' ].toSet()
rcaudy marked this conversation as resolved.
Show resolved Hide resolved
ProjectType type = ProjectType.getType(p)
if (!type.isPublic) {
return false
Expand Down
20 changes: 20 additions & 0 deletions clock-impl/build.gradle
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
plugins {
id 'io.deephaven.project.register'
}

description 'Implementation for the Clock service to obtain allocation-free precise time.'

dependencies {
implementation project(':clock')
Classpaths.inheritAutoService(project)
}

tasks.withType(JavaCompile).configureEach {
options.compilerArgs += ['--add-exports', 'java.base/jdk.internal.misc=ALL-UNNAMED']
// Explicitly set the source compatibility so gradle will invoke javac with `-source 11` instead of `--release`
sourceCompatibility = 11
}

tasks.withType(Javadoc).configureEach {
options.addStringOption('-add-exports', 'java.base/jdk.internal.misc=ALL-UNNAMED')
}
1 change: 1 addition & 0 deletions clock-impl/gradle.properties
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
io.deephaven.project.ProjectType=JAVA_PUBLIC
Loading