-
Notifications
You must be signed in to change notification settings - Fork 1k
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Checking mergeability…
Don’t worry, you can still create the pull request.
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: graphprotocol/graph-node
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: graphprotocol/graph-node
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: lutter/sql
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 13 commits
- 24 files changed
- 2 contributors
Commits on Mar 8, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 505401d - Browse repository at this point
Copy the full SHA 505401dView commit details -
Configuration menu - View commit details
-
Copy full SHA for f8743a7 - Browse repository at this point
Copy the full SHA f8743a7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8b50615 - Browse repository at this point
Copy the full SHA 8b50615View commit details -
Configuration menu - View commit details
-
Copy full SHA for a2dca57 - Browse repository at this point
Copy the full SHA a2dca57View commit details -
1. Do not use CTE's to inject a view of a table at a certain block. Instead rewrite the 'from' clause 2. Do not turn bytea columns into string columns since that is hugely wasteful
Configuration menu - View commit details
-
Copy full SHA for bb3850b - Browse repository at this point
Copy the full SHA bb3850bView commit details -
store: Extract SQL parsing tests into a YAML file
That setup makes it much easier to add more tests that check that we scrub dangerous constructs from SQL
Configuration menu - View commit details
-
Copy full SHA for 82d9ec7 - Browse repository at this point
Copy the full SHA 82d9ec7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 35bd6fe - Browse repository at this point
Copy the full SHA 35bd6feView commit details -
Configuration menu - View commit details
-
Copy full SHA for bd5817f - Browse repository at this point
Copy the full SHA bd5817fView commit details -
store: Defer wrapping of SQL queries until execution
The wrapping with to_jsonb is closely tied to how we run the query
Configuration menu - View commit details
-
Copy full SHA for 2b1435a - Browse repository at this point
Copy the full SHA 2b1435aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 84349ba - Browse repository at this point
Copy the full SHA 84349baView commit details -
Configuration menu - View commit details
-
Copy full SHA for 133cadb - Browse repository at this point
Copy the full SHA 133cadbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 804a208 - Browse repository at this point
Copy the full SHA 804a208View commit details
Commits on Mar 12, 2025
-
Configuration menu - View commit details
-
Copy full SHA for ef3ee18 - Browse repository at this point
Copy the full SHA ef3ee18View commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff master...lutter/sql