-
Notifications
You must be signed in to change notification settings - Fork 52
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 update-omd-discover-s3-asset.md to reflect #1818
Conversation
that some of the Custom Properties have been removed. Also, the default tag category is now GenericTags (replacing Fybrik) Signed-off-by: Doron Chen <[email protected]>
1. Register an asset through the OpenMetadata UI 2. Registering Dataset via Connector Signed-off-by: Doron Chen <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in the pictures you continue to use GenericTags
, and you have GenericTags.PII
and PII.Sensitive
Store the asset ID in a `CATALOGED_ASSET` variable: | ||
```bash | ||
CATALOGED_ASSET="openmetadata-s3.default.demo.\"PS_20174392719_1491204439457_log.csv\"" | ||
``` | ||
|
||
The asset is now registered in the catalog. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the OPA policies examples below are not updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest that we should address this in a different PR. I could work on it tomorrow morning.
<br><center><img src="../../static/openmetadata/18_table_name.jpg" width=600></center> | ||
1. Choose a tag for the dataset, such as `Fybrik.finance` | ||
1. Choose a tag for the dataset, such as `GenericTags.finance` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe Purpose.finance
Signed-off-by: Doron Chen <[email protected]>
that some of the Custom Properties have been
removed.
Also, the default tag category is now GenericTags
(replacing Fybrik)
Signed-off-by: Doron Chen [email protected]