ferruzzi commented on a change in pull request #22189:
URL: https://github.com/apache/airflow/pull/22189#discussion_r825002609
##########
File path:
airflow/providers/amazon/aws/example_dags/example_emr_job_flow_automatic_steps.py
##########
@@ -44,38 +45,40 @@
'InstanceGroups': [
{
'Name': 'Primary node',
- 'Market': 'SPOT',
+ 'Market': 'ON_DEMAND',
'InstanceRole': 'MASTER',
- 'InstanceType': 'm1.medium',
+ 'InstanceType': 'm5.xlarge',
Review comment:
Just a note since I'm sure someone will ask; These are the new
default/recommended settings. The m1 series instance types are "previous
generation" and no longer recommend.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
