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

transport: QueryInfo contents inaccessible outside of transport #287

Open
Kulezi opened this issue Sep 13, 2022 · 1 comment
Open

transport: QueryInfo contents inaccessible outside of transport #287

Kulezi opened this issue Sep 13, 2022 · 1 comment

Comments

@Kulezi
Copy link
Contributor

Kulezi commented Sep 13, 2022

QueryInfo contains only unexported fields:

// QueryInfo represents data required for host selection policy to create query plan.
// Token and strategy are only necessary for token aware policies.
type QueryInfo struct {
	tokenAware bool
	token      Token
	topology   *topology
	strategy   strategy
	offset     uint64 // For round robin strategies.
}

This creates a problem that its impossible to create a custom HostSelectionPolicy outside of transport package that would make any use of QueryInfo

@Kulezi
Copy link
Contributor Author

Kulezi commented Sep 13, 2022

Also wouldn't QueryInfo benefit from holding keyspace (which has a strategy) instead of strategy?

@Kulezi Kulezi changed the title transport: QueryInfo contents inaccessible transport: QueryInfo contents inaccessible outside of transport Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant