You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "github-actions[bot] (via GitHub)" <gi...@apache.org> on 2025/05/29 14:41:13 UTC

[PR] [v3-0-test] Update `TaskLogContent` to support virtualized rendering (#50746) [airflow]

github-actions[bot] opened a new pull request, #51202:
URL: https://github.com/apache/airflow/pull/51202

   * Update TaskLogContent to support virtualized rendering
   
   * Update TaskLogPreview and Logs to handle undefined parsedLogs
   (cherry picked from commit 813f3e3f15985c017b615b5e4a33ed9c4496fdbe)
   
   Co-authored-by: Guan Ming(Wesley) Chiu <10...@users.noreply.github.com>


-- 
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


Re: [PR] [v3-0-test] Update `TaskLogContent` to support virtualized rendering (#50746) [airflow]

Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on PR #51202:
URL: https://github.com/apache/airflow/pull/51202#issuecomment-2934501398

   I just rebased this PR on top to anticipate the CI.


-- 
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


Re: [PR] [v3-0-test] Update `TaskLogContent` to support virtualized rendering (#50746) [airflow]

Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil commented on PR #51202:
URL: https://github.com/apache/airflow/pull/51202#issuecomment-2934385200

   Could we fix this possibly in next 20-30 mins? If not, I will have to move this to 3.0.3


-- 
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


Re: [PR] [v3-0-test] Update `TaskLogContent` to support virtualized rendering (#50746) [airflow]

Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on PR #51202:
URL: https://github.com/apache/airflow/pull/51202#issuecomment-2934160503

   cc: @bbovenzi (merger)  @guan404ming (author)


-- 
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


Re: [PR] [v3-0-test] Update `TaskLogContent` to support virtualized rendering (#50746) [airflow]

Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil merged PR #51202:
URL: https://github.com/apache/airflow/pull/51202


-- 
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


Re: [PR] [v3-0-test] Update `TaskLogContent` to support virtualized rendering (#50746) [airflow]

Posted by "guan404ming (via GitHub)" <gi...@apache.org>.
guan404ming commented on PR #51202:
URL: https://github.com/apache/airflow/pull/51202#issuecomment-2934422564

   This type error happened since the upstream PR https://github.com/apache/airflow/pull/50547 is not cherry picked here. Should we add it in or just fix the static here. If so, I think I currently don't have permission to push to this branch. But change is simple is just in `useLogs.tsx`:
   
   ```
   - accept?: "*/*" | "application/json" | "application/x-ndjson";
   + accept?: "*/*" | "application/json" | "text/plain";
   ```
   
   
   maybe someone has permission could help with this!


-- 
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


Re: [PR] [v3-0-test] Update `TaskLogContent` to support virtualized rendering (#50746) [airflow]

Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on PR #51202:
URL: https://github.com/apache/airflow/pull/51202#issuecomment-2934490910

   Needed backport PR first: https://github.com/apache/airflow/pull/51357


-- 
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


Re: [PR] [v3-0-test] Update `TaskLogContent` to support virtualized rendering (#50746) [airflow]

Posted by "guan404ming (via GitHub)" <gi...@apache.org>.
guan404ming commented on PR #51202:
URL: https://github.com/apache/airflow/pull/51202#issuecomment-2934495693

   Thanks!


-- 
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