-
Notifications
You must be signed in to change notification settings - Fork 334
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
tracking issue for refactoring the Table
trait and related structures
#2065
Labels
Comments
waynexia
added
C-enhancement
Category Enhancements
tracking-issue
A tracking issue for a feature.
labels
Aug 2, 2023
waynexia
changed the title
refactor the
tracking issue for refactoring the Aug 2, 2023
Table
trait and related structuresTable
trait and related structures
2 tasks
This was referenced Aug 15, 2023
This was referenced Aug 24, 2023
This was referenced Sep 4, 2023
Close as completed |
I'm trying to drop the Table trait ... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
What type of enhancement is this?
Tech debt reduction
What does the enhancement do?
With table engine in datanode being replaced by region engine in #1869, we should also reconsider the
Table
abstraction. Generally speaking, the datanode (or region server) will no longer be aware of table -- it only sees and handles regions. And consequently, all the data access kind API (read, write, delete etc.) should be removed fromTable
. This makes theTable
trait only in response of metadata, like retrieving table info or table schema.This change will be shipped in the following parts:
TableInfo
feat: prepare for implementing considering partition key in the distributed planner #2000TableFactory
PartitionExec
, which is used to serveDistTable
'sscan()
method*Adaptor
s and*Adapter
s style: merge adaptor and adapter #1665Table
trait with the newTable
struct refactor(table): remove Table impl for system #2270Implementation challenges
No response
The text was updated successfully, but these errors were encountered: