Skip to content

Unnecessary re-render due to unclear database query sorting #1

@satkowski

Description

@satkowski

The core sqlseal plugin sorts the files in an (at least for now) unclear way. I would currently assume that the order of the query results are based on last file opening or edit. However, this causes the comparison if the old rendered table is the same as the new one, to fail unnecessarily, since the content didn't change, but only the order. This can be seen in the following example (only the order of Materializer Test - General.md and Materializer Test - Links.md changed):

plugin:sqlseal-materalizer:334 < OLD:
| id                                           | name                           |
| -------------------------------------------- | ------------------------------ |
| Materializer Test - General.md               | Materializer Test - General    |
| Materializer Test - Links.md                 | Materializer Test - Links      |
| Materializer Test - Images.md                | Materializer Test - Images     |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes |
| Weekly/2024-W51.md                           | 2024-W51                       |
| Basics.md                                    | Basics                         |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             |
| Weekly/2024-W01.md                           | 2024-W01                       |
| Tags.md                                      | Tags                           |
plugin:sqlseal-materalizer:336 > NEW:
| id                                           | name                           |
| -------------------------------------------- | ------------------------------ |
| Materializer Test - General.md               | Materializer Test - General    |
| Materializer Test - Links.md                 | Materializer Test - Links      |
| Materializer Test - Images.md                | Materializer Test - Images     |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes |
| Weekly/2024-W51.md                           | 2024-W51                       |
| Basics.md                                    | Basics                         |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             |
| Weekly/2024-W01.md                           | 2024-W01                       |
| Tags.md                                      | Tags                           |
plugin:sqlseal-materalizer:338 = ???: true
plugin:sqlseal-materalizer:307 MD-PRINT Renderer: Table is the same, not updating.


plugin:sqlseal-materalizer:334 < OLD:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:336 > NEW:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:338 = ???: false
plugin:sqlseal-materalizer:320 MD-PRINT Renderer: Table printed/updated.


plugin:sqlseal-materalizer:334 < OLD:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:336 > NEW:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:338 = ???: true
plugin:sqlseal-materalizer:307 MD-PRINT Renderer: Table is the same, not updating.


plugin:sqlseal-materalizer:334 < OLD:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:336 > NEW:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:338 = ???: false
plugin:sqlseal-materalizer:320 MD-PRINT Renderer: Table printed/updated.


plugin:sqlseal-materalizer:334 < OLD:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:336 > NEW:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:338 = ???: true
plugin:sqlseal-materalizer:307 MD-PRINT Renderer: Table is the same, not updating.


plugin:sqlseal-materalizer:334 < OLD:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:336 > NEW:
| id                                           | name                           | a(path, name)                                                         |
| -------------------------------------------- | ------------------------------ | --------------------------------------------------------------------- |
| Materializer Test - Links.md                 | Materializer Test - Links      | [[Materializer Test - Links.md\|Materializer Test - Links]]           |
| Materializer Test - General.md               | Materializer Test - General    | [[Materializer Test - General.md\|Materializer Test - General]]       |
| Materializer Test - Images.md                | Materializer Test - Images     | [[Materializer Test - Images.md\|Materializer Test - Images]]         |
| Materializer Test - Checkboxes.md            | Materializer Test - Checkboxes | [[Materializer Test - Checkboxes.md\|Materializer Test - Checkboxes]] |
| Weekly/2024-W51.md                           | 2024-W51                       | [[Weekly/2024-W51.md\|2024-W51]]                                      |
| Basics.md                                    | Basics                         | [[Basics.md\|Basics]]                                                 |
| Showcase/01 Recursive CTE/SQLite Examples.md | SQLite Examples                | [[Showcase/01 Recursive CTE/SQLite Examples.md\|SQLite Examples]]     |
| Welcome to SQLSeal.md                        | Welcome to SQLSeal             | [[Welcome to SQLSeal.md\|Welcome to SQLSeal]]                         |
| Weekly/2024-W01.md                           | 2024-W01                       | [[Weekly/2024-W01.md\|2024-W01]]                                      |
| Tags.md                                      | Tags                           | [[Tags.md\|Tags]]                                                     |
plugin:sqlseal-materalizer:338 = ???: true
plugin:sqlseal-materalizer:307 MD-PRINT Renderer: Table is the same, not updating.

With that, the way I compare the new and the old table needs to be changed in the future.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions