yujunjun/datahub
2
0
Fork 0
mirror of https://github.com/datahub-project/datahub.git synced 2025-11-07 06:13:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(GraphQL): Add schemaMetadata mapping & correctly form path string (#2084)

Browse Source 
* Fixing DataHub frontend Browse bug

* Add schema mapping

Co-authored-by: John Joyce <john@acryl.io>
This commit is contained in:
John Joyce 2021-02-05 12:56:50 -08:00 committed by GitHub
parent b73723b98c
commit b0801de7ad
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/types/dataset/DatasetType.java
View File

@ -107,8 +107,9 @@ public class DatasetType implements SearchableEntityType<Dataset>, BrowsableEnti
int start, int start,
int count) throws Exception { int count) throws Exception {
final Map<String, String> facetFilters = ResolverUtils.buildFacetFilters(filters, FACET_FIELDS); final Map<String, String> facetFilters = ResolverUtils.buildFacetFilters(filters, FACET_FIELDS);
final String pathStr = path.size() > 0 ? BROWSE_PATH_DELIMITER + String.join(BROWSE_PATH_DELIMITER, path) : "";
final BrowseResult result = _datasetsClient.browse( final BrowseResult result = _datasetsClient.browse(
BROWSE_PATH_DELIMITER + String.join(BROWSE_PATH_DELIMITER, path), pathStr,
facetFilters, facetFilters,
start, start,
count); count);

1
datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/types/mappers/DatasetMapper.java
View File

@ -42,6 +42,7 @@ public class DatasetMapper implements ModelMapper<com.linkedin.dataset.Dataset,
if (dataset.getSchemaMetadata().hasDeleted()) { if (dataset.getSchemaMetadata().hasDeleted()) {
result.setDeleted(AuditStampMapper.map(dataset.getSchemaMetadata().getDeleted())); result.setDeleted(AuditStampMapper.map(dataset.getSchemaMetadata().getDeleted()));
} }
result.setSchema(SchemaMetadataMapper.map(dataset.getSchemaMetadata()));
} }
if (dataset.hasPlatformNativeType()) { if (dataset.hasPlatformNativeType()) {
result.setPlatformNativeType(Enum.valueOf(PlatformNativeType.class, dataset.getPlatformNativeType().name())); result.setPlatformNativeType(Enum.valueOf(PlatformNativeType.class, dataset.getPlatformNativeType().name()));