-
Notifications
You must be signed in to change notification settings - Fork 12
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
Initial version of the HTTP API + DF upgrade #31
Commits on Jul 27, 2022
-
Configuration menu - View commit details
-
Copy full SHA for 8fc9577 - Browse repository at this point
Copy the full SHA 8fc9577View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7caaf22 - Browse repository at this point
Copy the full SHA 7caaf22View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4bac231 - Browse repository at this point
Copy the full SHA 4bac231View commit details -
Configuration menu - View commit details
-
Copy full SHA for d6fd956 - Browse repository at this point
Copy the full SHA d6fd956View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4be7e90 - Browse repository at this point
Copy the full SHA 4be7e90View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6b500ac - Browse repository at this point
Copy the full SHA 6b500acView commit details -
Run the PG/HTTP frontends selectively in
main
Because we prepare a vector of all futures (PG/HTTP frontends), there's a lifetime issue with the config structs that I don't completely understand (they basically need to live as long as the `'static` lifetime). We can't pass them by reference to the PG/HTTP frontend because that reference might not live as long as static, since we're doing this as a separate function that's called from main (?)). Our `prepare_frontends` function can't own the config either (it has to outlive it). To fix this, we pass the config to the PG/HTTP frontends by-value and clone it.
Configuration menu - View commit details
-
Copy full SHA for d7aa069 - Browse repository at this point
Copy the full SHA d7aa069View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2fbfb7f - Browse repository at this point
Copy the full SHA 2fbfb7fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8a7d056 - Browse repository at this point
Copy the full SHA 8a7d056View commit details -
Break the
SeafowlContext
code into a traitThis is used so that we can mock SeafowlContext easier in unit tests.
Configuration menu - View commit details
-
Copy full SHA for 5616e42 - Browse repository at this point
Copy the full SHA 5616e42View commit details -
Configuration menu - View commit details
-
Copy full SHA for b5b7ece - Browse repository at this point
Copy the full SHA b5b7eceView commit details -
Configuration menu - View commit details
-
Copy full SHA for e6c62a1 - Browse repository at this point
Copy the full SHA e6c62a1View commit details -
Configuration menu - View commit details
-
Copy full SHA for c4dde4c - Browse repository at this point
Copy the full SHA c4dde4cView commit details -
Add unit tests for the HTTP GET endpoint
(this mocking is getting really painful)
Configuration menu - View commit details
-
Copy full SHA for b14d13f - Browse repository at this point
Copy the full SHA b14d13fView commit details -
Fix test failures related to the DF upgrade
- Parquet hashes changed (probably fine, due to changes in ArrowWriter / DF's execution output?) - Output of `SHOW TABLES` changed (new column)
Configuration menu - View commit details
-
Copy full SHA for c2f2d74 - Browse repository at this point
Copy the full SHA c2f2d74View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6177769 - Browse repository at this point
Copy the full SHA 6177769View commit details -
Configuration menu - View commit details
-
Copy full SHA for 87abbc1 - Browse repository at this point
Copy the full SHA 87abbc1View commit details -
Configuration menu - View commit details
-
Copy full SHA for dff760b - Browse repository at this point
Copy the full SHA dff760bView commit details -
Use an actual config /
build_context
in e2e testsThis is to exercise the actual context creation code and remove code duplication
Configuration menu - View commit details
-
Copy full SHA for c4112ea - Browse repository at this point
Copy the full SHA c4112eaView commit details