Skip to content

Commit 1d31f92

Browse files
authored
chore: return columns order info (#35)
* chore: return columns order info * upgrade ubuntu from 20.04 to 22.04 --------- Co-authored-by: rick <[email protected]>
1 parent 7e3a936 commit 1d31f92

File tree

4 files changed

+20
-9
lines changed

4 files changed

+20
-9
lines changed

.github/workflows/build.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ on:
55

66
jobs:
77
Test:
8-
runs-on: ubuntu-20.04
8+
runs-on: ubuntu-22.04
99
steps:
1010
- name: Set up Go
1111
uses: actions/setup-go@v3
@@ -24,7 +24,7 @@ jobs:
2424
bash <(curl -Ls https://coverage.codacy.com/get.sh) final
2525
2626
Build:
27-
runs-on: ubuntu-20.04
27+
runs-on: ubuntu-22.04
2828
steps:
2929
- name: Set up Go
3030
uses: actions/setup-go@v3
@@ -38,14 +38,14 @@ jobs:
3838
args: release --clean --snapshot
3939

4040
BuildImage:
41-
runs-on: ubuntu-20.04
41+
runs-on: ubuntu-22.04
4242
steps:
4343
- uses: actions/[email protected]
4444
- name: Image
4545
run: make build-image
4646

4747
RunE2E:
48-
runs-on: ubuntu-20.04
48+
runs-on: ubuntu-22.04
4949
steps:
5050
- uses: actions/[email protected]
5151
- name: Run e2e

.github/workflows/release-drafter.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ on:
77

88
jobs:
99
UpdateReleaseDraft:
10-
runs-on: ubuntu-20.04
10+
runs-on: ubuntu-22.04
1111
steps:
1212
- uses: release-drafter/release-drafter@v5
1313
env:

.github/workflows/release.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ env:
1515

1616
jobs:
1717
Test:
18-
runs-on: ubuntu-20.04
18+
runs-on: ubuntu-22.04
1919
steps:
2020
- name: Set up Go
2121
uses: actions/setup-go@v3
@@ -34,7 +34,7 @@ jobs:
3434
bash <(curl -Ls https://coverage.codacy.com/get.sh) final
3535
3636
goreleaser:
37-
runs-on: ubuntu-20.04
37+
runs-on: ubuntu-22.04
3838
if: github.ref != 'refs/heads/master'
3939
steps:
4040
- name: Checkout
@@ -70,7 +70,7 @@ jobs:
7070
oras push ${{ env.REGISTRY_ALIYUN }}/linuxsuren/atest-ext-store-orm:${TAG#v} */*
7171
7272
image:
73-
runs-on: ubuntu-20.04
73+
runs-on: ubuntu-22.04
7474
steps:
7575
- name: Checkout
7676
uses: actions/[email protected]

pkg/data_query.go

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ package pkg
1818
import (
1919
"context"
2020
"database/sql"
21+
"encoding/json"
2122
"fmt"
2223
"log"
2324
"reflect"
@@ -80,7 +81,7 @@ func (s *dbserver) Query(ctx context.Context, query *server.DataQuery) (result *
8081

8182
wg.Add(1)
8283
go func() {
83-
wg.Done()
84+
defer wg.Done()
8485
result.Meta.Labels = dbQuery.GetLabels(ctx, query.Sql)
8586
result.Meta.Labels = append(result.Meta.Labels, &server.Pair{
8687
Key: "_native_sql",
@@ -93,6 +94,9 @@ func (s *dbserver) Query(ctx context.Context, query *server.DataQuery) (result *
9394
if dataResult, err = sqlQuery(ctx, query.Sql, db); err == nil {
9495
result.Items = dataResult.Items
9596
result.Meta.Duration = time.Since(now).String()
97+
98+
wg.Wait()
99+
result.Meta.Labels = append(result.Meta.Labels, dataResult.Meta.Labels...)
96100
}
97101
return
98102
}
@@ -128,6 +132,13 @@ func sqlQuery(ctx context.Context, sql string, db *gorm.DB) (result *server.Data
128132
return
129133
}
130134

135+
if columnsData, colsErr := json.Marshal(columns); colsErr == nil {
136+
result.Meta.Labels = append(result.Meta.Labels, &server.Pair{
137+
Key: "_columns",
138+
Value: string(columnsData),
139+
})
140+
}
141+
131142
for rows.Next() {
132143
// Create a slice of interface{}'s to represent each column,
133144
// and a second slice to contain pointers to each item in the columns slice.

0 commit comments

Comments
 (0)