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.

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

feat: add event tables to sdk #2215

Merged
merged 8 commits into from
Dec 8, 2023
Merged

feat: add event tables to sdk #2215

merged 8 commits into from
Dec 8, 2023

Conversation

sfc-gh-swinkler
Copy link
Collaborator


// CreateEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/create-event-table.
type CreateEventTableOptions struct {
create bool `ddl:"static" sql:"CREATE"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field create is unused

type CreateEventTableOptions struct {
create bool `ddl:"static" sql:"CREATE"`
OrReplace *bool `ddl:"keyword" sql:"OR REPLACE"`
eventTable bool `ddl:"static" sql:"EVENT TABLE"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field eventTable is unused


// ShowEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/show-event-tables.
type ShowEventTableOptions struct {
show bool `ddl:"static" sql:"SHOW"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field show is unused

// ShowEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/show-event-tables.
type ShowEventTableOptions struct {
show bool `ddl:"static" sql:"SHOW"`
eventTables bool `ddl:"static" sql:"EVENT TABLES"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field eventTables is unused


// DescribeEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/describe-event-table.
type DescribeEventTableOptions struct {
describe bool `ddl:"static" sql:"DESCRIBE"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field describe is unused


// AlterEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/alter-event-table.
type AlterEventTableOptions struct {
alter bool `ddl:"static" sql:"ALTER"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field alter is unused

// AlterEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/alter-event-table.
type AlterEventTableOptions struct {
alter bool `ddl:"static" sql:"ALTER"`
eventTable bool `ddl:"static" sql:"TABLE"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field eventTable is unused

}

type EventTableDropRowAccessPolicy struct {
rowAccessPolicy bool `ddl:"static" sql:"ROW ACCESS POLICY"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field rowAccessPolicy is unused

}

type SearchOptimization struct {
searchOptimization bool `ddl:"static" sql:"SEARCH OPTIMIZATION"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field searchOptimization is unused

return opts
}

func (r eventTableRow) convert() *EventTable {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
func eventTableRow.convert is unused

Copy link

Integration tests failure for 3ac856ad207507859ab81dab506ed8161769fbda

pkg/sdk/event_tables_def.go Outdated Show resolved Hide resolved
pkg/sdk/event_tables_def.go Outdated Show resolved Hide resolved
pkg/sdk/event_tables_def.go Show resolved Hide resolved
pkg/sdk/event_tables_def.go Outdated Show resolved Hide resolved
pkg/sdk/event_tables_def.go Outdated Show resolved Hide resolved
pkg/sdk/event_tables_gen_test.go Show resolved Hide resolved
pkg/sdk/event_tables_impl_gen.go Show resolved Hide resolved
pkg/sdk/event_tables_validations_gen.go Outdated Show resolved Hide resolved
pkg/sdk/testint/event_tables_integration_test.go Outdated Show resolved Hide resolved
pkg/sdk/event_tables_def.go Outdated Show resolved Hide resolved
pkg/sdk/event_tables_def.go Outdated Show resolved Hide resolved
pkg/sdk/event_tables_def.go Show resolved Hide resolved
pkg/sdk/event_tables_def.go Outdated Show resolved Hide resolved
pkg/sdk/event_tables_def.go Outdated Show resolved Hide resolved
pkg/sdk/event_tables_def.go Show resolved Hide resolved
pkg/sdk/testint/event_tables_integration_test.go Outdated Show resolved Hide resolved

// ShowEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/show-event-tables.
type ShowEventTableOptions struct {
show bool `ddl:"static" sql:"SHOW"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field show is unused

// ShowEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/show-event-tables.
type ShowEventTableOptions struct {
show bool `ddl:"static" sql:"SHOW"`
eventTables bool `ddl:"static" sql:"EVENT TABLES"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field eventTables is unused


// DropEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/drop-event-table.
type DropEventTableOptions struct {
drop bool `ddl:"static" sql:"DROP"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field drop is unused

// DropEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/drop-event-table.
type DropEventTableOptions struct {
drop bool `ddl:"static" sql:"DROP"`
table bool `ddl:"static" sql:"TABLE"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field table is unused


// AlterEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/alter-event-table.
type AlterEventTableOptions struct {
alter bool `ddl:"static" sql:"ALTER"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field alter is unused

// AlterEventTableOptions is based on https://docs.snowflake.com/en/sql-reference/sql/alter-event-table.
type AlterEventTableOptions struct {
alter bool `ddl:"static" sql:"ALTER"`
table bool `ddl:"static" sql:"TABLE"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field table is unused

}

type EventTableAddRowAccessPolicy struct {
add bool `ddl:"static" sql:"ADD"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field add is unused

}

type EventTableDropRowAccessPolicy struct {
drop bool `ddl:"static" sql:"DROP"`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
field drop is unused

return t
}

func (r *DescribeEventTableRequest) toOpts() *DescribeEventTableOptions {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [staticcheck] <U1000> reported by reviewdog 🐶
func (*DescribeEventTableRequest).toOpts is unused

Copy link

Integration tests failure for f68a196cb7bddc86da86fe03007844d4fb8bd63e

Copy link

Integration tests failure for 6d26e1e6379bc60ca99ec6cb3e5572ce85c1f1ce

pkg/sdk/event_tables_def.go Show resolved Hide resolved
pkg/sdk/event_tables_def.go Show resolved Hide resolved
pkg/sdk/event_tables_def.go Show resolved Hide resolved
pkg/sdk/event_tables_def.go Outdated Show resolved Hide resolved
Name().
WithValidation(g.ValidIdentifier, "name"),
).DropOperation(
"https://docs.snowflake.com/en/sql-reference/sql/drop-event-table",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it should point to the drop table page (there's no such page as drop event table)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

Copy link

github-actions bot commented Dec 5, 2023

Integration tests failure for 38fffb91a9dcee635def5460e7ec0246dcbeffaf

Copy link

github-actions bot commented Dec 5, 2023

Integration tests failure for 9f2e0bd082b9b972068cc506cc5d36cd1165a208

Copy link

github-actions bot commented Dec 5, 2023

Integration tests failure for c884a41259b4d218f832a780d30f73d5f7706065

@sfc-gh-jcieslak sfc-gh-jcieslak mentioned this pull request Dec 5, 2023
17 tasks
Copy link

github-actions bot commented Dec 7, 2023

Integration tests failure for 5d68d59bb599b3e38d52ecb4fa8624c41398dfca

@sfc-gh-swinkler sfc-gh-swinkler merged commit 66cc80a into main Dec 8, 2023
5 of 7 checks passed
@sfc-gh-swinkler sfc-gh-swinkler deleted the event-tables-sdk branch December 8, 2023 07:24
Copy link

github-actions bot commented Dec 8, 2023

Integration tests failure for b43974480aa4bad94d350d1ec09dc5aa35ea7ec2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants