Skip to content

Update julia set example comments#4167

Merged
asfgit merged 1 commit intoapache:masterfrom
aaltay:julia
Nov 22, 2017
Merged

Update julia set example comments#4167
asfgit merged 1 commit intoapache:masterfrom
aaltay:julia

Conversation

@aaltay
Copy link
Copy Markdown
Member

@aaltay aaltay commented Nov 22, 2017

Copy link
Copy Markdown
Contributor

@charlesccychen charlesccychen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this LGTM.

# ['apt-get', '--assume-yes', 'install', 'libjpeg62'],
#
# First, note that there is no need to use the sudo command because the setup
# script runs with appropriate access.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: this line is Dataflow-specific until the FnAPI / containerization effort lands.

# ['apt-get', '--assume-yes', 'install', 'libjpeg62'],
#
# First, note that there is no need to use the sudo command because the setup
# script runs with appropriate access.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

charlesccychen wrote:
Note: this line is Dataflow-specific until the FnAPI / containerization effort lands.

Acknowledged. This needs some additional clean up.

@asfgit asfgit merged commit 7fe7732 into apache:master Nov 22, 2017
asfgit pushed a commit that referenced this pull request Nov 22, 2017
@aaltay aaltay deleted the julia branch January 19, 2018 23:38
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