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

[NO MERGE] PoC Dagger DI as a system builder #8411

Draft
wants to merge 28 commits into
base: master
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
a73606c
Add EventChannelSubscriber
Nashatyrev Jun 5, 2024
330b4cd
Narrow down parameter from EventChannels to EventChannelSubscriber
Nashatyrev Jun 7, 2024
e73895d
Initial version of Dagger modules (no Dagger compile check)
Nashatyrev Jun 7, 2024
a7639ae
Add RecentChainData async creation and initialization. Add WSModule
Nashatyrev Jun 19, 2024
40acba3
Complete BeaconChainController Dagger draft
Nashatyrev Jun 20, 2024
4815f66
Fix compile errors
Nashatyrev Jun 20, 2024
b100903
Add BeaconChainControllerComponent
Nashatyrev Jun 20, 2024
0938622
Resolve missing and duplicate bindings
Nashatyrev Jun 20, 2024
6434467
Resolve circular dependencies
Nashatyrev Jun 21, 2024
f5b05a9
New Dagger based BeaconChainController
Nashatyrev Jun 21, 2024
259935a
Make old/new BeaconChainController easily interchangeable
Nashatyrev Jun 21, 2024
1dd893e
Initialize BeaconChainController Dagger components inside doStart() w…
Nashatyrev Jun 21, 2024
6f7bd8c
Start AsyncRunnerEventThread right after creation
Nashatyrev Jun 21, 2024
a7eb722
Fix the mess with currentTime & genesisTime parameters order for getC…
Nashatyrev Jun 21, 2024
b067905
WeakSubjectivityFinalizedConfig updates storage prior to RecentChainD…
Nashatyrev Jun 21, 2024
34059a0
Add missing ValidatorApiChannel subscription
Nashatyrev Jun 24, 2024
6bd40ff
!!! TO ROLLBACK !!! Temp remove -Werror option
Nashatyrev Jun 25, 2024
8d491ef
Fix compiler warnings
Nashatyrev Jun 26, 2024
7277cc5
Add stricter Dagger compiler options
Nashatyrev Jun 26, 2024
8c5ee39
Return back -Werror
Nashatyrev Jun 26, 2024
b7893a7
Enable errorprone disableWarningsInGeneratedCode
Nashatyrev Jun 26, 2024
f569747
Spotless
Nashatyrev Jun 26, 2024
e8c0296
Remove AbstractBeaconChainController
Nashatyrev Jun 26, 2024
a8958fb
Redesign BeaconChainController creation
Nashatyrev Jun 26, 2024
c99e740
Adopt TekuConfigurationTest
Nashatyrev Jun 26, 2024
8ed7f5d
Merge branch 'refs/heads/master' into experiment/dagger-beacon-chain
Nashatyrev Jun 27, 2024
04e8b3b
Adopt latest config PRs to master
Nashatyrev Jul 1, 2024
124292c
Made method params final
Nashatyrev Jul 1, 2024
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
Prev Previous commit
Next Next commit
Make old/new BeaconChainController easily interchangeable
  • Loading branch information
Nashatyrev committed Jun 24, 2024
commit 259935a31a90a1a35971bc6d13e617a3dc046b8e
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
package tech.pegasys.teku.services.beaconchain;

import tech.pegasys.teku.service.serviceutils.Service;

public abstract class AbstractBeaconChainController extends Service implements BeaconChainControllerFacade {

}
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
* initialization behavior (see {@link BeaconChainControllerFactory}} however this class may change
* in a backward incompatible manner and either break compilation or runtime behavior
*/
public class BeaconChainController extends Service implements BeaconChainControllerFacade {
public class BeaconChainController extends AbstractBeaconChainController {

private static final Logger LOG = LogManager.getLogger();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@
* initialization behavior (see {@link BeaconChainControllerFactory}} however this class may change
* in a backward incompatible manner and either break compilation or runtime behavior
*/
public class BeaconChainControllerOld extends Service implements BeaconChainControllerFacade {
public class BeaconChainControllerOld extends AbstractBeaconChainController {

private static final Logger LOG = LogManager.getLogger();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@

public class BeaconChainService extends Service implements BeaconChainServiceFacade {

private final BeaconChainController controller;
private final AbstractBeaconChainController controller;

public BeaconChainService(
final ServiceConfig serviceConfig, final BeaconChainConfiguration beaconConfig) {
Expand All @@ -38,7 +38,7 @@ protected SafeFuture<?> doStop() {
}

@Override
public BeaconChainController getBeaconChainController() {
public BeaconChainControllerFacade getBeaconChainController() {
return controller;
}
}