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

Update base.py #30

Merged
merged 6 commits into from
Nov 24, 2023
Merged

Update base.py #30

merged 6 commits into from
Nov 24, 2023

Conversation

janne123456789
Copy link
Contributor

Update name of keyword argument for the call to self.client.raw.tables.create() from 'table' to 'name', which is corresponding to cognite-sdk versions ^6.0.0.

janne123456789 and others added 4 commits October 24, 2023 20:15
Update name of keyword argument for the call to self.client.raw.tables.create() from 'table' to 'name', which is corresponding to cognite-sdk versions ^6.0.0.
- switch to `extra="forbid"`
- which avoids unnoticed misspelled properties in configurations
- like Dict, Set and Tuple
@spex66
Copy link
Collaborator

spex66 commented Oct 29, 2023

- expected it to be visble as "Owner" column in extpipe listing
- but it isn't, maybe a Fusion bug?
- changing the processing
  - which wasn't working when `naming-pattern` where used
  - using ExtractionPipelineList and `as_external_ids()` now
@spex66
Copy link
Collaborator

spex66 commented Oct 29, 2023

@janne123456789
Good catch with the v6 parameter!

I made several fixes now and tested

  • adding/removal/adding
  • with and without raw tables
  • with and without "automatic delete"

But with naming-pattern config only (there were more bugs than the raw-table parameter)

Someone can test the changes please with your config?

Then we can release it as v3.0.0-beta2
(See commits for step wise changes)

@spex66 spex66 requested a review from pavelzubarev October 29, 2023 14:29
Copy link
Collaborator

@spex66 spex66 left a comment

Choose a reason for hiding this comment

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

lgtm
but I made more changes (bugs and features)
so I appreciate another one run it with your config

  1. add an extpipe
  2. remove an extpipe
  3. with and without raw-tables

worked for me using naming-patterns

@spex66 spex66 merged commit db3705f into main Nov 24, 2023
2 checks passed
@spex66 spex66 deleted the janne123456789-create-raw-table-fix branch November 24, 2023 08:46
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.

2 participants