Add a MLMD context for every `workspace` & update notebook example

zhenghuiwang commented 4 years ago

What this PR does / why we need it: workspace is for group relevant metadata together. MLMD recently added generic Context&ContextType for the same purpose.

This PR adds one context for each workspace to enable fast metadata retrieval via context. Current way of adding workspace name as artifact property shall be gradually phased out in the future.(Current UI still needs it)

Context name is required to be unique for different contexts. This PR adds the option of whether reusing context with same name when initializing the workspace.

Changes:

Make python tests idempotent. Update notebook test setup to use SDK HEAD.
Update demo.ipynb notebook with the latest SDK.
Update README, setup.py and __init__ to prepare SDK release.

This change is

zhenghuiwang commented 4 years ago

/retest

zhenghuiwang commented 4 years ago

@jinchihe and @gaoning777 This PR is ready for your review! thanks

jinchihe commented 4 years ago

@zhenghuiwang Thanks. Sounds great!

/lgtm

zhenghuiwang commented 4 years ago

Thanks for the review! /approve

k8s-ci-robot commented 4 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: zhenghuiwang

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/kubeflow/metadata/blob/master/OWNERS)~~ [zhenghuiwang] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

kubeflow / metadata

Add a MLMD context for every `workspace` & update notebook example #178