-
Notifications
You must be signed in to change notification settings - Fork 623
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
CASSGO-29 refactor packages to separate frontend api to backend internal api #465
Comments
A good starting point would be to move the unexported marshalling code into an internal package and export just the Marshall / Unmarshall functions from the gocql package |
@joao-r-reis I can handle it, but there are a few tricky moments:
What do you think? |
I think the project is not that big that it needs a lot of packages, the main purpose of this ticket should be to at least keep the public API stuff on the current |
Maybe we could add an internal protocol package for framer and marshal code right away too (maybe a "native protocol package") |
Oh keep in mind that since this will change a lot of files it will be hard to keep the PR in sync with all other PRs that are waiting to be merged atm. There will be a lot of work involved in resolving git conflicts. |
Ok, I'll start with the framer, and then we can check it as an example |
It might make more sense to group framer and native protocol stuff in a single package but I'm not sure |
Currently everything exists within the gocql package, as the driver functionality expands the package gets more and more complicated.
We should maintain the current API in the gocql package, but move much of the internal things out into sub packages.
The text was updated successfully, but these errors were encountered: