Skip to content

Master thesis merge #134

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 156 commits into from
Dec 4, 2024
Merged

Master thesis merge #134

merged 156 commits into from
Dec 4, 2024

Conversation

@AlanRostem
Copy link
Collaborator

No description provided.

AlanRostem and others added 30 commits June 6, 2024 16:48
@AlanRostem
Some struct field name changes in core.go and a type reflection schem…
2e49221 
…e for adding pipelined modules
@AlanRostem
Test case for AddPipelined(), but it is incomplete. Passes without an…
7fe8655 
…y panics
@AlanRostem
Pipelined module test considers multiple modules. Still incomplete, b…
6cd47f1 
…ut compiles with no errors
@AlanRostem
Added a type check to AddPipelined in core.go
75aa424
@AlanRostem
Testing out a new pipelined module creation scheme
4a12276
@AlanRostem
Builder now builds pipelined modules to Core. All tests pass without …
11b9792 
…any panics.
@AlanRostem
Test case for module pipelines done
d18e7e2
@AlanRostem
Fixed a bug in multiplier test
2110d11
@AlanRostem
Created a TryGet for pipelined modules which do something similar but…
567c3e0 
… accept nil ptrs. Test case is basic and doesn't panic, but more cases are needed.
@AlanRostem
Bug in TryGetPipelined fixed where nil value check was inverted. Adde…
b07cb00 
…d a test case for pipelined modules, which passes
@AlanRostem
Code cleanup and additional documentation
8b0fe92
@AlanRostem
Added helpers to enable creating pipelines from ids specified by the …
5f302d0 
…user.
@AlanRostem
Changed function naming to minimize needed refactorings. Added commen…
8081129 
…ts and documentation with examples. Second test case was also added to test pipelined functions in builder when pipelining is disabled. All tests pass.
@AlanRostem
Fixed a potential bug in builder.Build(). Corrections to error messag…
014d4f8 
…es added. Test Passing.
@AlanRostem
Renamed the pipelined get functions in core.go
d66dc23
@AlanRostem
Separated pipeline ID type and constants to another go-module so that…
d84daed 
… other go-modules can access pipelining-related stuff.
@AlanRostem
Created a separate pipelined event loop to avoid code refactorings. T…
47f4057 
…est case created for handling event emissions from unique pipelines. Passes.
@AlanRostem
Pipelined eventloop passes the same type of tests as the regular one.…
4f83a51 
… Missing benchmarks.
@AlanRostem
Optimized pipelined eventloop maps that hold handlers and tickers
50572b0
@AlanRostem
Added standard function versions to PipelinedEventLoop. Next step is …
5a11a2b 
…to integrate this version to the rest of the codebase.
@AlanRostem
New branch for network-related pipelining integration.
9744f29
@AlanRostem
Eventloop supporting pipelining integrated to all of the code.
1fb2afe
@AlanRostem
Experimental RPC for pipelined proposals only
bb15725
@AlanRostem
Refactoring words like 'pipelined' to 'piped'. ProposePiped reverted …
3fcb234 
…back to take one message.
@AlanRostem
Remaining refactoring from previous commit
70cdf51
@AlanRostem
Got a new RPC to work
2dba180
@AlanRostem
Remaining rpcs added. Need to change eventloop again
2647ebf
@AlanRostem
Refined the eventloop's pipelining capabilities. Some validation adde…
8370b37 
…d to pipelining in core.
@AlanRostem
Merge pull request #1 from AlanRostem/network-pipelining
8bcfc9f 
Merging network-pipelining to master.
@AlanRostem
Cleaner pipelining in eventloop. InitModule takes in a pipeId now.
90a6f54
AlanRostem and others added 29 commits November 5, 2024 15:23
@AlanRostem
Cleaning up orchestration.
b168ecb
@AlanRostem
Some file cleanups and old test fixes
d87bfd5
@AlanRostem
Merge branch 'master' of github.com:AlanRostem/hotstuff-pipelined
5f06f3b
@AlanRostem
Hacky latency induction is being done on a different goroutine.
03435ed
@AlanRostem
Rename committer modules and added them to their own package.
8aef8c6
@AlanRostem
Adding pipeline-awareness to some modules and added a few todo's
81b2358
@AlanRostem
Merge resolved.
1cab647
@AlanRostem
Forgot the rest of the server.go file.
75170ad
@AlanRostem
Renamed pipe value to instance and all references to pipe are renamed…
32f6080 
… to refer to a consensus instance verbatim.
@AlanRostem
Renaming pipes to instance in protobuf definitions.
b84e616
@AlanRostem
Rename RespondToPipe to RespondToInstance.
6c96899
@AlanRostem
Removed junk files
380be52
@AlanRostem
Renaming pipe -> scope
3a55811
@AlanRostem
Bug occurs when using unscoped eventloop in client and measurements. …
4c9824c 
…Removed its usage.
@AlanRostem
Remaining name changes
6c4a401
@AlanRostem
Forgot to save this one file.
6cf634a
@AlanRostem
Cleaned up launch config and further renaming.
610d5e6
@AlanRostem
Using name multiplexed committer.
81c06a3
@AlanRostem
Further naming fixes.
5a57a64
@AlanRostem
Rename InitOptions -> ScopeInfo
2da2ad6
@AlanRostem
Going back to using pipe as a name
42d8198
@AlanRostem
Committer rename
554fb59
@AlanRostem
Rename more accurate
5f249ef
@AlanRostem
Slight rename to adding scoped modules.
e8455e4
@AlanRostem
Small name change to a variable in block.Bytes()
3412bc8
@AlanRostem
Fixed bugs with creating fixed vs dynamic view duration
619ee52
@AlanRostem
Forgot to fix one consensus test
02c30ae
@AlanRostem
Disabled TLS to prevent a bug. Added debug lines to replica stopping …
0ab81a2 
…since it fails.
@AlanRostem
Update README.md
ae4dccc
@leandernikolaus leandernikolaus merged commit 6f9699c into relab:pipelining-alan-rostem Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AlanRostem @leandernikolaus