Skip to content

Fix metadata tables that use manifestListLocation#803

Merged
danielcweeks merged 3 commits into
apache:masterfrom
rdblue:fix-manifest-list-location
Feb 17, 2020
Merged

Fix metadata tables that use manifestListLocation#803
danielcweeks merged 3 commits into
apache:masterfrom
rdblue:fix-manifest-list-location

Conversation

@rdblue

@rdblue rdblue commented Feb 16, 2020

Copy link
Copy Markdown
Contributor

Some metadata tables assume that manifestListLocation is set, but this is an optional file that may not have been written in older tables.

@danielcweeks

Copy link
Copy Markdown
Contributor

+1

@danielcweeks danielcweeks merged commit ad925bd into apache:master Feb 17, 2020
@aokolnychyi

Copy link
Copy Markdown
Contributor

I think this also resolves #552.

@rdblue

rdblue commented Feb 21, 2020

Copy link
Copy Markdown
Contributor Author

@aokolnychyi, you're right. Originally the fix was in #801, but the two changes conflicted so I fixed both problems in this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants