Skip to content

Proposal: group querier interface by query filenames or tags in query annotations #1594

@yiplee

Description

@yiplee

What do you want to change?

group by query filenames

// grouped by query file
type Querier interface {
	// property.sql
        UpdateProperty(ctx context.Context, arg UpdatePropertyParams) (sql.Result, error)
        SaveProperty(ctx context.Context, arg SavePropertyParams) error
        // user.sql
	SaveUser(ctx context.Context, arg SaveUserParams) error
        // asset.sql
        SaveAsset(ctx context.Context, arg SaveAssetParams) error
	UpdateAsset(ctx context.Context, arg UpdateAssetParams) (sql.Result, error)
}

group by tags in query annotations

-- name: UpdateProperty :execresult group:Property
Update properties SET `value` =  ? WHERE `key` =  ?;
type Querier interface {
	// Property
        UpdateProperty(ctx context.Context, arg UpdatePropertyParams) (sql.Result, error)
}

What database engines need to be changed?

MySQL

What programming language backends need to be changed?

Go

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions