[GitHub] [airflow] ephraimbuddy commented on issue #29405: Add pagination to get_log in the rest API

["ephraimbuddy (via GitHub)" <gi...@apache.org>]

ephraimbuddy commented on issue #29405:
URL: https://github.com/apache/airflow/issues/29405#issuecomment-1455706930

   > @ephraimbuddy when I looked into the code in log_endpoint.py, the full_content is a boolean that decides how to populate the metadata. with that value set in metadata only, the download_logs is set/unset in the metadata. can you tell me why a similar solution won't work log_pos?
   
   I don't think we need the log_pos, it's not a boolean like the download_logs. It's something that should come from the logger.
   At the first request, we can just get the log with its position. It seems to me that giving it position would mean truncating some logs


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org