Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

AccountantSkel -> Tpu #188

Merged
merged 2 commits into from
May 9, 2018
Merged

AccountantSkel -> Tpu #188

merged 2 commits into from
May 9, 2018

Conversation

garious
Copy link
Contributor

@garious garious commented May 8, 2018

The terms Stub and Skel come from OMG IDL and only made sense while
the Stub was acting as an RPC client for the the Accountant object.
Nowadays, the Stub interface looks nothing like the Accountant and
meanwhile we've recognized the multithreaded implementation is more
reminiscent of a pipelined CPU. Thus, we finally bite the bullet and
rename our modules.

AccountantSkel -> Tpu
AccountantStub -> ThinClient

Up next will be moving much of the TPU code into separate modules,
each representing a stage of the pipeline. The interface of each
will follow the precedent set by the Historian object.

@garious garious requested review from sakridge and aeyakovenko May 8, 2018 23:39
The terms Stub and Skel come from OMG IDL and only made sense while
the Stub was acting as an RPC client for the the Accountant object.
Nowadays, the Stub interface looks nothing like the Accountant and
meanwhile we've recognized the multithreaded implementation is more
reminiscent of a pipelined CPU. Thus, we finally bite the bullet and
rename our modules.

AccountantSkel -> Tpu
AccountantStub -> ThinClient

Up next will be moving much of the TPU code into separate modules,
each representing a stage of the pipeline. The interface of each
will follow the precedent set by the Historian object.
@garious garious merged commit 3236be7 into solana-labs:master May 9, 2018
steviez pushed a commit to steviez/solana that referenced this pull request Mar 12, 2024
willhickey pushed a commit that referenced this pull request Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant