The GitHub Actions job "Tests" on airflow.git has failed.
Run started by GitHub user dstandish (triggered by dstandish).

Head commit for run:
64a88ddfe3beb93d5dfb368027656aa4e48249e8 / Daniel Standish 
<[email protected]>
Use None instead of empty data structures when no ElasticSearch logs

Previously, when there were no logs, we would just use [] or {} and continue to 
process them through functions such as "group by host" and such.  But if no 
logs are found, it's odd to go down those paths.

One example of the confusion is the return type in `es_read` was `list | 
ElasticSearchResponse`.  This makes it look like it would either be a list 
(presumably of logs?) or an ElasticSearchResponse object (presumably also 
containing logs?).  But in reality, the only time list was returned, was when 
there _were no logs at all!_  This was just to adhere to the contract that we 
handle "no logs" the same as "logs".  _(And it turns out that 
ElasticSearchResponse magically also behaves like at iterable which is also 
confusing but that's a separate issue.)_

Report URL: https://github.com/apache/airflow/actions/runs/6425116780

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to