Skip to content

sqlite#619

Merged
lixinqi merged 9 commits into
PaddlePaddle:developfrom
Honglei-Qiu:sqlite
Feb 3, 2026
Merged

sqlite#619
lixinqi merged 9 commits into
PaddlePaddle:developfrom
Honglei-Qiu:sqlite

Conversation

@Honglei-Qiu

Copy link
Copy Markdown
Contributor

PR Category

Feature Enhancement

Description

完成Repo GraphSample SubgraphSource DimensionGeneralizationSource表的插入,DataTypeGeneralizationSource暂时未操作,不知道具体格式

@paddle-bot

paddle-bot Bot commented Jan 29, 2026

Copy link
Copy Markdown

Thanks for your contribution!

Comment thread sqlite/graphnet.sql Outdated
@@ -0,0 +1,81 @@
-- SQLite
-- create Repo table
DROP TABLE IF EXISTS Repo;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

这个很危险啊

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

这个脚本一旦重复执行,数据就会立刻丢掉

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已修改

Comment thread sqlite/graphnet.sql Outdated
Comment on lines +21 to +22
repo_uid VARCHAR(255) NOT NULL,
relative_model_path TEXT NOT NULL UNIQUE,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

relative_model_path单独应该不是unique的。(relative_model_path, repo_uid)合起来才是。

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

每个表的联合索引都需要小心处理,区分度大的放前面

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已修改,现在在写ORM

Comment thread sqlite/graphnet.sql Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

重命名成create_main_tables_$(date +"%Y-%m-%d-%H%M%S").sql

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已修改

@Honglei-Qiu

Honglei-Qiu commented Jan 30, 2026

Copy link
Copy Markdown
Contributor Author

BackwardGraphSource 后续再改进,而且这个要确定最后的结果是对子图进行的backward存储还是整图的backward也要存储

@lixinqi

lixinqi commented Jan 30, 2026

Copy link
Copy Markdown
Collaborator

BackwardGraphSource 后续再改进,而且这个要确定最后的结果是对子图进行的backward存储还是整图的backward也要存储

可以

Comment thread sqlite/graphsample_insert.py Outdated


# DimensionGeneralizationSource insert func
def insert_DimensionGeneralizationSource(

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

这些函数名字太诡异了

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

insert_dimension_info ?
insert_datatype_info

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

insert_dimension_generalization_source

@Honglei-Qiu Honglei-Qiu Jan 31, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done,目前完成数据库的init和insert功能,下周继续完成数据库的delete功能,updata功能待定,同时添加“运行insert的sh功能”时产生时间戳log,表示在哪时候插入,init产生时间戳log,都加一个时间戳log,后续可以分析时间戳log来获得某些额外信息
这样可以替代create_main_tables_$(date +"%Y-%m-%d-%H%M%S").sql功能
示例:python ./sqlite/orm_models.py init | tee "sqlite/logs/init_$(date +'%Y-%m-%d-%H%M%S').log"

@lixinqi lixinqi merged commit 2335af3 into PaddlePaddle:develop Feb 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants