Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Implemented Sort Operator; Basic Sort query works #5

Merged
merged 1 commit into from
Feb 26, 2024
Merged

Conversation

SarveshOO7
Copy link
Collaborator

Note: Execplan must be removed from physical plans for basic_sort.sql

… Execplan must be removed from physical plans for basic_sort.sql
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 0% with 75 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (e025452) to head (622841a).
Report is 1 commits behind head on main.

Files Patch % Lines
eggstrain/src/execution/operators/sort.rs 0.00% 59 Missing ⚠️
eggstrain/src/execution/query_dag.rs 0.00% 12 Missing ⚠️
eggstrain/src/execution/operators/filter.rs 0.00% 4 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff          @@
##            main      #5   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          5       6    +1     
  Lines        242     311   +69     
  Branches     242     311   +69     
=====================================
- Misses       242     311   +69     

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

@ktbooker
Copy link
Collaborator

LGTM

@ktbooker ktbooker merged commit 04a41ca into main Feb 26, 2024
2 checks passed
@ktbooker ktbooker deleted the sort-operator branch February 26, 2024 00:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants