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

record number相关问题 #29

Closed
zhaoyiping0622 opened this issue Nov 17, 2020 · 14 comments · Fixed by #28
Closed

record number相关问题 #29

zhaoyiping0622 opened this issue Nov 17, 2020 · 14 comments · Fixed by #28
Assignees
Labels
documentation Improvements or additions to documentation question Further information is requested

Comments

@zhaoyiping0622
Copy link
Collaborator

我发现sqlite好多操作都是直接将record number压栈,我们还要不要保留record number这东西。
如果保留的话,dbengine需要给我增加接口。
如果不保留的话,每次压栈一个记录有可能会有性能损耗,以及之后可能会碰到一些其他问题(我猜的)。

@zhaoyiping0622 zhaoyiping0622 added documentation Improvements or additions to documentation question Further information is requested labels Nov 17, 2020
@zhaoyiping0622
Copy link
Collaborator Author

例如 idx相关 list相关IR 都是操作record number的
另外,IR中也有一些与record number相关的指令,例如NewRecno/recno,这些我暂时都还没实现,如果要实现的话,需要dbengine提供接口

@zhaoyiping0622 zhaoyiping0622 linked a pull request Nov 17, 2020 that will close this issue
@QingQiz
Copy link
Owner

QingQiz commented Nov 18, 2020

record number 是个什么东西啊

@zhaoyiping0622
Copy link
Collaborator Author

大概就是 数据库里每一行都有一个编号 叫做record number

@QingQiz
Copy link
Owner

QingQiz commented Nov 18, 2020

需要增加啥接口

@zhaoyiping0622
Copy link
Collaborator Author

获取一个recordnumber
通过recordnumber获取value
暂时只想得到这些

@zhaoyiping0622
Copy link
Collaborator Author

第一个指的是 获取一个新的record number

@zhaoyiping0622
Copy link
Collaborator Author

感觉recno好像没啥用

@QingQiz
Copy link
Owner

QingQiz commented Nov 18, 2020

我也感觉没啥用,,

@QingQiz
Copy link
Owner

QingQiz commented Nov 18, 2020

这玩意感觉没有好像也没啥问题

@zhaoyiping0622
Copy link
Collaborator Author

不过感觉删了这玩意
好像很多IR都要改了

@zhaoyiping0622
Copy link
Collaborator Author

那个makeidxkey就可以直接删了

@zhaoyiping0622
Copy link
Collaborator Author

不过这个recordnumber可以唯一标示一条数据 不知道这个有没有什么应用场景

@zhaoyiping0622
Copy link
Collaborator Author

那就先删了?
感觉IR好多要修改了

@QingQiz
Copy link
Owner

QingQiz commented Nov 18, 2020

行,删了吧

@QingQiz QingQiz closed this as completed Nov 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants