Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

disallow column longer then 63 characters #9

Closed
yevgenypats opened this issue Apr 4, 2021 · 2 comments
Closed

disallow column longer then 63 characters #9

yevgenypats opened this issue Apr 4, 2021 · 2 comments
Assignees
@zagronitay
Labels
enhancement New feature or request

Comments

@yevgenypats
Copy link
Member

No description provided.

@roneli roneli added the enhancement New feature or request label Apr 18, 2021
@roneli
Copy link
Contributor

roneli commented Apr 22, 2021 •
edited
Loading

Verify also tables names are no longer than NAMEDATALEN - 1 i.e 63 bytes

@roneli roneli mentioned this issue Apr 22, 2021
Closed
@roneli
Copy link
Contributor

roneli commented May 3, 2021

  • Verify columns names
  • Verify table names

zagronitay added a commit to zagronitay/cq-provider-sdk that referenced this issue May 3, 2021
@zagronitay
Added resource set validation cloudquery#7
7ca9ac8 
Added TableValidation to enforce column/table name longer than 63 char cloudquery#9
@zagronitay zagronitay mentioned this issue May 3, 2021
Merged
zagronitay added a commit to zagronitay/cq-provider-sdk that referenced this issue May 4, 2021
@zagronitay
Fixed CR
f8f2df7 
Added resource set validation cloudquery#7

Added TableValidation to enforce column/table name longer than 63 char cloudquery#9
zagronitay added a commit to zagronitay/cq-provider-sdk that referenced this issue May 4, 2021
@zagronitay
Fixed CR
79c0cdc 
Added resource set validation cloudquery#7

Added TableValidation to enforce column/table name longer than 63 char cloudquery#9
roneli pushed a commit that referenced this issue May 4, 2021
@zagronitay
Added resource set validation (#20)
7d8a9a3 
* Added resource set validation #7
* Added TableValidation to enforce column/table name longer than 63 char #9
@roneli roneli closed this as completed May 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@zagronitay zagronitay

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@zagronitay @yevgenypats @roneli