Skip to content

Conversation

@Pangjiping
Copy link
Collaborator

Summary

  • Add a custom Scanner.Split that treats both \n and \r as delimiters (handling \r\n as one) to capture progress outputs, and raise the max token size to 5MB to avoid truncating long lines.
  • Add unit tests covering CR/LF splitting, incremental reads, and long-line handling.

Testing

  • Not run (explain why)
  • Unit tests
  • Integration tests
  • e2e / manual verification

Breaking Changes

  • None
  • Yes (describe impact and migration path)

Checklist

  • Linked Issue or clearly described motivation
  • Added/updated docs (if needed)
  • Added/updated tests (if needed)
  • Security impact considered
  • Backward compatibility considered

Copy link
Collaborator

@hittyt hittyt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hittyt hittyt merged commit dd03abb into alibaba:main Dec 22, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants