Skip to content
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.

Sign up for GitHub

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

Pushdown simplest sorting for aggregate command #2530

Merged
merged 26 commits into from
May 3, 2023
Merged

Pushdown simplest sorting for aggregate command #2530

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
e13337b
wip
noisersup Apr 24, 2023
12d2d3d
wip
noisersup Apr 25, 2023
a31ebe7
wip
noisersup Apr 25, 2023
787b05a
wip
noisersup Apr 26, 2023
00d97d9
wip
noisersup Apr 26, 2023
78605d0
wip
noisersup Apr 27, 2023
5b97891
wip
noisersup Apr 27, 2023
281bed0
wip
noisersup Apr 28, 2023
8770a28
Merge remote-tracking branch 'upstream/main' into pushdown-sort-2438
noisersup Apr 28, 2023
e3a2333
wip
noisersup Apr 28, 2023
256ee3b
wip
noisersup Apr 28, 2023
47baa37
wip
noisersup Apr 28, 2023
773fb94
Merge branch 'main' into pushdown-sort-2438
noisersup Apr 28, 2023
0883777
Merge branch 'main' into pushdown-sort-2438
mergify[bot] Apr 28, 2023
d34e056
wip
noisersup Apr 28, 2023
e5ffaea
wip
noisersup Apr 28, 2023
5c93458
wip
noisersup Apr 28, 2023
ae05028
wip
noisersup Apr 28, 2023
50497e1
wip
rumyantseva May 2, 2023
ccfbe61
wip
rumyantseva May 2, 2023
79c9dd1
wip
rumyantseva May 2, 2023
40c99dd
wip
rumyantseva May 2, 2023
ebbe58f
wip
rumyantseva May 2, 2023
2013ab4
wip
rumyantseva May 3, 2023
329306e
wip
rumyantseva May 3, 2023
a81592e
Merge branch 'main' into pushdown-aggregation-sort-2438
May 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge remote-tracking branch 'upstream/main' into pushdown-sort-2438
  • Loading branch information
@noisersup
noisersup committed Apr 28, 2023
commit 8770a28e37e262b16e835ca42d2ec11a89701b5c
16 changes: 13 additions & 3 deletions internal/handlers/pg/msg_aggregate.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -189,9 +189,19 @@ func processStagesDocuments(ctx context.Context, p *stagesDocumentsParams) ([]*t
var docs []*types.Document

if err := p.dbPool.InTransaction(ctx, func(tx pgx.Tx) error {
iter, _, getErr := pgdb.QueryDocuments(ctx, tx, p.qp)
if getErr != nil {
return getErr
var err error
iter, _, err := pgdb.QueryDocuments(ctx, tx, p.qp)
if err != nil {
return err
}

closer := iterator.NewMultiCloser(iter)
defer closer.Close()

for _, s := range p.stages {
if iter, err = s.Process(ctx, iter, closer); err != nil {
return err
}
}

docs, err = iterator.ConsumeValues(iterator.Interface[struct{}, *types.Document](iter))
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.