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

add Keysets to to table dto / info #452

Merged
merged 7 commits into from
Aug 24, 2021
Merged

add Keysets to to table dto / info #452

merged 7 commits into from
Aug 24, 2021

Conversation

zcxcasd
Copy link
Contributor

@zcxcasd zcxcasd commented Aug 20, 2021

adding keysets to table dto + info for jdbc connector tables

sample output from http:https://localhost:7001/mds/v1/catalog/nrdportal/database/nrdportalprod/table/bf_cert_projects like below:

"keys":
{
    "partition":
    [],
    "primary":
    [
        {
            "name": "PRIMARY",
            "fields":
            [
                "id"
            ]
        }
    ],
    "sort":
    [],
    "index":
    []
},

@NoArgsConstructor
@AllArgsConstructor
@EqualsAndHashCode(callSuper = false)
public class KeyInfo extends BaseDto {
Copy link
Contributor

Choose a reason for hiding this comment

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

Use BaseInfo

@zcxcasd zcxcasd changed the title Keysets add Keysets to to table dto / info Aug 24, 2021
@ajoymajumdar ajoymajumdar added this to the 1.3.0 milestone Aug 24, 2021
@zcxcasd zcxcasd merged commit 6bb1a68 into Netflix:master Aug 24, 2021
ajoymajumdar added a commit to ajoymajumdar/metacat that referenced this pull request Sep 17, 2021
ajoymajumdar added a commit that referenced this pull request Sep 17, 2021
rmendybayev added a commit to rmendybayev/metacat that referenced this pull request Sep 20, 2021
* Avoid setting prevous location to current location for Iceberg table …

* Remove unused imports to fix check style (Netflix#455)

*  Logging for Iceberg table commits (Netflix#457)

* Revert "add Keysets to to table dto / info  (Netflix#452)" (Netflix#458)

* Integration of Netflix Metacat and Netflix Data Explorer

* Metacat on k8s

* Integration of Netflix Metacat and Netflix Data Explorer

* Integration of Netflix Metacat and Netflix Data Explorer

* Integration of Netflix Metacat and Netflix Data Explorer

* Merge remote-tracking branch 'origin/epam-mvplatform' into epam-mvpla…

* Metacat on k8s

* Added new github action for GCR build

* Updated gitignore

* Turning on push to GCR on epam-mvplatform (main branch of this fork t…

* Merge remote-tracking branch 'origin/epam-mvplatform' into epam-mvpla…

* Revert "Integration of Netflix Metacat and Netflix Data Explorer - Fi…

* Swagger UI fixed
zcxcasd added a commit to zcxcasd/metacat that referenced this pull request Sep 20, 2021
zcxcasd added a commit that referenced this pull request Sep 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants