Trying to save a Trial (SQA) with TTL seconds fails

[chasewalden]

It seems the SQATrial definition may be broken. It looks like a copy-paste bug from the two lines above.

Ax/ax/storage/sqa_store/sqa_classes.py

Lines 398 to 399 in 1a62e09

	# pyre-fixme[8]: Attribute has type `Optional[int]`; used as `Column[bool]`.
	ttl_seconds: Optional[int] = Column(Boolean)

Causes this issue when saving to SQL:

(builtins.ValueError) Value 10 is not None, True, or False
[SQL: INSERT INTO trial_v2 (abandoned_reason, deployed_name, experiment_id, \"index\", is_batched, num_arms_created, optimize_for_power, ttl_seconds, run_metadata, status, status_quo_name, time_completed, time_created, time_staged, time_run_started, trial_type, generation_step_index, properties) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)]\n[parameters: [{'run_metadata': {}, 'is_batched': False, 'ttl_seconds': 10, 'index': 0, 'status': <TrialStatus.CANDIDATE: 0>, 'properties': {}, 'generation_step_ind ... (160 characters truncated) ... ed': None, 'trial_type': None, 'time_completed': None, 'deployed_name': None, 'status_quo_name': None, 'time_staged': None, 'abandoned_reason': None}]]

[stevemandala]

Thanks for catching this bug! bf15d40 should resolve this and is on master now. Should get included in the next release (which should be soon).