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

Reduce raft deps #18043

Merged
merged 4 commits into from
Apr 25, 2024
Merged

Conversation

mmaslankaprv
Copy link
Member

@mmaslankaprv mmaslankaprv commented Apr 24, 2024

Raft types.h file contains a lot of definitions that are raft internal. Now that we have raft/fundamentals.h we can remove the dependency on types.h file

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.1.x
  • v23.3.x
  • v23.2.x

Release Notes

  • none

Move basic named type aliases from `metadata.h` to `fundamental.h`.
There is not clear separation between the two but the metadata file
contains less common and more complex types. It should be enough to
include the `fundamental.h` for most of the cases.

Signed-off-by: Michał Maślanka <michal@redpanda.com>
Previously we included `raft/types.h` in every generated RPC service
stub. This isn't now required as we moved the definition of
transfer leadership related types to separate file.

Signed-off-by: Michał Maślanka <michal@redpanda.com>
rockwotj
rockwotj previously approved these changes Apr 24, 2024
Copy link
Contributor

@rockwotj rockwotj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

Signed-off-by: Michał Maślanka <michal@redpanda.com>
Raft `types.h` file contains a lot of definitions that are raft
internal. Now that we have `raft/fundamentals.h` we can remove the
dependency on `types.h` file

Signed-off-by: Michał Maślanka <michal@redpanda.com>
Copy link
Member

@BenPope BenPope left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

@vbotbuildovich
Copy link
Collaborator

vbotbuildovich commented Apr 24, 2024

new failures in https://buildkite.com/redpanda/redpanda/builds/48208#018f0ffd-f7f5-4ad2-b8a2-7f824dff1853:

"rptest.tests.rbac_upgrade_test.UpgradeMigrationCreatingDefaultRole.test_rbac_migration"

new failures in https://buildkite.com/redpanda/redpanda/builds/48208#018f102a-b8b5-49f0-a224-3679950e5d4e:

"rptest.tests.rbac_upgrade_test.UpgradeMigrationCreatingDefaultRole.test_rbac_migration"

@mmaslankaprv
Copy link
Member Author

/ci-repeat 1

@mmaslankaprv mmaslankaprv merged commit 7fd9f10 into redpanda-data:dev Apr 25, 2024
12 of 18 checks passed
@mmaslankaprv mmaslankaprv deleted the reduce-raft-deps branch April 25, 2024 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants