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

feat: update ml system UI #12334

Merged
merged 57 commits into from
Jan 29, 2025
Merged
Changes from 1 commit
Commits
Show all changes
57 commits
Select commit Hold shift + click to select a range
7008a23
[wip] update ui changes
yoonhyejin Jan 14, 2025
6148597
fix typo
yoonhyejin Jan 14, 2025
bbcc2d4
add mlmodellineageinfo
yoonhyejin Jan 14, 2025
963b060
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 14, 2025
8dd3c25
update mapper for lineageinfo and add uni tests
yoonhyejin Jan 15, 2025
2046691
[wip] add script for creating mlmodel
yoonhyejin Jan 15, 2025
a5f9a3f
spotlessapply
yoonhyejin Jan 15, 2025
054bc1f
feat(graphql/versioning): Add versioning support to graphql; mutation…
asikowitz Jan 15, 2025
06bc8c1
add back fragments.graphql
yoonhyejin Jan 15, 2025
5a6923c
fix tests
RyanHolstien Jan 16, 2025
69da53a
Merge remote-tracking branch 'andrew-fork/versioning-graphql' into fe…
yoonhyejin Jan 16, 2025
314b869
fix fragments.graphql
yoonhyejin Jan 16, 2025
43e3d07
[wip] fix ui bugs
yoonhyejin Jan 16, 2025
e365b06
fix rendering issues
yoonhyejin Jan 17, 2025
88db564
add run event in script
yoonhyejin Jan 17, 2025
b62ef19
update script and add islinegae:True back
yoonhyejin Jan 17, 2025
b5066c6
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 17, 2025
d4dc426
add mlModel to DPIinput relationships entityTypes
yoonhyejin Jan 20, 2025
31dd659
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 20, 2025
3c1c3b7
fix container dpi preview
yoonhyejin Jan 21, 2025
cdc23a8
spotlessapply
yoonhyejin Jan 21, 2025
8a92827
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 21, 2025
0209b7e
fix prefect test
yoonhyejin Jan 21, 2025
2adba1d
fix linting
yoonhyejin Jan 21, 2025
fe1e9cd
fix linting
yoonhyejin Jan 21, 2025
757b914
fix yarn lint
yoonhyejin Jan 21, 2025
c4999be
fix ruff check
yoonhyejin Jan 21, 2025
7a3bdf5
[wip] fix linting
yoonhyejin Jan 21, 2025
f7336bc
fix linting
yoonhyejin Jan 21, 2025
a4c600c
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 22, 2025
da7aee9
update golden files for dbt & fivetran
yoonhyejin Jan 22, 2025
534e3b4
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 22, 2025
01f17c3
update golden files for airflow
yoonhyejin Jan 22, 2025
24a9070
update golden files for dags
yoonhyejin Jan 22, 2025
28359e3
fix airflow tests to include dataplatforminstance
yoonhyejin Jan 22, 2025
43fe5e2
fix linitng
yoonhyejin Jan 22, 2025
0501a2b
hacky fix for search filters test
shirshanka Jan 22, 2025
398450c
fix searchfilters cypress test
shirshanka Jan 22, 2025
8cdc27d
some golden file updates
shirshanka Jan 23, 2025
c52a462
Add argparser for token (#6)
ryota-cloud Jan 23, 2025
b198047
update goldens
shirshanka Jan 23, 2025
e48a7a6
fix preview entity
yoonhyejin Jan 23, 2025
426d80c
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 23, 2025
0d99464
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 24, 2025
2f4aaea
remove commented out code
yoonhyejin Jan 24, 2025
9f7f298
fix empty ml model message
yoonhyejin Jan 24, 2025
946854c
remove console logs
yoonhyejin Jan 24, 2025
fa713df
pr feedback pair programming
asikowitz Jan 28, 2025
a1d48c2
revert metadata-ingestion/ & metadata-ingestion-modules/ files to head
yoonhyejin Jan 28, 2025
8a42b7e
revert some of the changes manually
yoonhyejin Jan 28, 2025
a33ef73
revert some of the changes manually
yoonhyejin Jan 28, 2025
d4af718
update stylings
yoonhyejin Jan 28, 2025
07adba4
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 28, 2025
e43ed52
Merge branch 'master' into feat/update-mlflow-ui
yoonhyejin Jan 28, 2025
a1c0125
revert search filters
asikowitz Jan 28, 2025
dfd66bf
fix properties.name mapping
yoonhyejin Jan 28, 2025
2b9a418
fix yarnlint
yoonhyejin Jan 29, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix tests
RyanHolstien committed Jan 16, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
commit 5a6923cf66dfe7de1f3fecbedb4d967da1a3d65e
Original file line number Diff line number Diff line change
@@ -11,6 +11,7 @@
import com.linkedin.datahub.graphql.QueryContext;
import com.linkedin.datahub.graphql.featureflags.FeatureFlags;
import com.linkedin.datahub.graphql.generated.LinkVersionInput;
import com.linkedin.datahub.graphql.generated.VersionSet;
import com.linkedin.metadata.entity.IngestResult;
import com.linkedin.metadata.entity.versioning.EntityVersioningService;
import com.linkedin.metadata.entity.versioning.VersionPropertiesInput;
@@ -56,8 +57,8 @@ public void testGetSuccessful() throws Exception {
Mockito.when(mockEnv.getArgument(Mockito.eq("input"))).thenReturn(input);
Mockito.when(mockEnv.getContext()).thenReturn(mockContext);

String result = resolver.get(mockEnv).get();
assertEquals(result, TEST_ENTITY_URN);
VersionSet result = resolver.get(mockEnv).get();
assertEquals(result.getUrn(), TEST_ENTITY_URN);
}

@Test
Original file line number Diff line number Diff line change
@@ -48,7 +48,7 @@ public void testGetSuccessful() throws Exception {
Mockito.when(mockEnv.getArgument(Mockito.eq("input"))).thenReturn(input);
Mockito.when(mockEnv.getContext()).thenReturn(mockContext);

assertTrue(resolver.get(mockEnv).get());
assertNotNull(resolver.get(mockEnv).get());

Mockito.verify(mockService)
.unlinkVersion(
Original file line number Diff line number Diff line change
@@ -52,7 +52,7 @@ public class VersionsSearchResolverTest {

private static final SearchAcrossEntitiesInput BASIC_INPUT =
new SearchAcrossEntitiesInput(
List.of(EntityType.DATASET), "", 0, 10, null, null, null, null, null, null, null, null);
List.of(EntityType.DATASET), "", 0, 10, null, null, null, null, null);

private static final SearchAcrossEntitiesInput COMPLEX_INPUT =
new SearchAcrossEntitiesInput(
@@ -99,10 +99,7 @@ public class VersionsSearchResolverTest {
.setField("sortField2")
.setSortOrder(SortOrder.ASCENDING)
.build()))
.build(),
null,
null,
null);
.build());

@Test
public void testGetSuccessBasic() throws Exception {