Fixing _update_max_id_per_time for SQLGraph

[yfukai]

Fixing issue with _update_max_id_per_time after saving the SQLite, and test this.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.36%. Comparing base (e87a375) to head (cd33d22).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
+ Coverage   88.29%   88.36%   +0.07%     
==========================================
  Files          53       54       +1     
  Lines        3809     3843      +34     
  Branches      661      663       +2     
==========================================
+ Hits         3363     3396      +33     
- Misses        266      267       +1     
  Partials      180      180              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[JoOkuma]

Hey @yfukai, I didn't realize at the time, but I think we can remove the loop. Would a groupby operation work?

stmt = (
    select(self.Node.t, func.max(self.Node.node_id))
    .group_by(self.Node.t)
)

[yfukai]

Hi, thanks for the suggestion. I updated the implementation using groupby!