Skip to content

Add index to object_ids array in splitgraph_meta.tables#542

Merged
gruuya merged 9 commits intomasterfrom
authz/keto-integration-CU-1ac3xfj
Sep 30, 2021
Merged

Add index to object_ids array in splitgraph_meta.tables#542
gruuya merged 9 commits intomasterfrom
authz/keto-integration-CU-1ac3xfj

Conversation

@gruuya
Copy link
Copy Markdown
Contributor

@gruuya gruuya commented Sep 27, 2021 •
edited
Loading

A couple of minor changes relevant to the new authorization logic:

  • Add index to object_ids array in splitgraph_meta.tables
  • Make init_fdw accept custom role for setting up FDW user mapping.
  • Set search path to public for a couple of splitgraph_api functions
  • ClickUp hook cu-1ac3xfj

@gruuya gruuya changed the title Customize FDW user mapping role Add index to object_ids array in splitgraph_meta.tables Sep 27, 2021
@gruuya gruuya marked this pull request as ready for review September 28, 2021 06:59
@gruuya gruuya requested a review from mildbyte September 28, 2021 06:59
@gruuya gruuya self-assigned this Sep 28, 2021
mildbyte
mildbyte reviewed Sep 28, 2021
View reviewed changes
Comment thread splitgraph/hooks/data_source/fdw.py Outdated
:param wrapper: Name of the foreign data wrapper (must be installed as an extension on the engine)
:param server_options: Dictionary of FDW options
:param user_options: Dictionary of user options
:param role: The name of the roll for which the user mapping is created; defaults to public.
Copy link
Copy Markdown
Contributor

@mildbyte mildbyte Sep 28, 2021

Choose a reason for hiding this comment

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

minor: name of the role

@gruuya gruuya merged commit f62dc13 into master Sep 30, 2021
@gruuya gruuya deleted the authz/keto-integration-CU-1ac3xfj branch September 30, 2021 11:08
mildbyte added a commit that referenced this pull request Oct 14, 2021
@mildbyte
Bump version: 0.2.16 → 0.2.17
3e13088 
  * Code refactor / optimizations (#531)
  * Support for pluggable authorization logic (#542, #549)
  * FDW JSONSchema fixes (#545)
  * Upgrade pglast to 3.4 to fix issues with Splitfile validation (#534)
  * Speed up Splitfile builds (#550)

Full set of changes: [`v0.2.16...v0.2.17`](v0.2.16...v0.2.17)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mildbyte mildbyte mildbyte approved these changes

Assignees

@gruuya gruuya

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@gruuya @mildbyte