Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Docker build should not remove heron-core.tar.gz #3291

Merged
merged 1 commit into from
Jun 17, 2019
Merged

Docker build should not remove heron-core.tar.gz #3291

merged 1 commit into from
Jun 17, 2019

Conversation

windhamwong
Copy link
Contributor

The file /usr/local/heron/dist/heron-core.tar.gz is used during heron submit command. This file should not be removed during docker build, otherwise heron submit will not succeed.

@nwangtw
Copy link
Contributor

nwangtw commented Jun 17, 2019

KK. Make sense. It was removed to reduce image size, but "docker run" was used for testing instead of "heron submit". This change makes sense. We will find another way to reduce size later.

Thanks.

@nwangtw nwangtw merged commit e167cec into apache:master Jun 17, 2019
sreev pushed a commit to sreev/incubator-heron that referenced this pull request Apr 9, 2020
nicknezis pushed a commit that referenced this pull request Sep 14, 2020
sreev pushed a commit to sreev/incubator-heron that referenced this pull request Nov 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants