Skip to content

Conversation

@Akanksha-kedia
Copy link
Contributor

Description

Fixed 4 common typos found throughout the codebase:

  1. serialziedserialized (comment)
  2. deserialzeFromdeserializeFrom (method name)
  3. seperatorseparator (variable name)
  4. occuredoccurred (comment)

@Akanksha-kedia
Copy link
Contributor Author

@xiangfu0 please help to review

@codecov-commenter
Copy link

codecov-commenter commented Oct 28, 2025

Codecov Report

❌ Patch coverage is 40.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.27%. Comparing base (549a47c) to head (1a697bb).
⚠️ Report is 20 commits behind head on master.

Files with missing lines Patch % Lines
...ata/manager/realtime/RealtimeTableDataManager.java 0.00% 2 Missing ⚠️
...ealtime/writer/StatelessRealtimeSegmentWriter.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17094      +/-   ##
============================================
- Coverage     63.47%   63.27%   -0.20%     
- Complexity     1424     1428       +4     
============================================
  Files          3086     3104      +18     
  Lines        182340   183349    +1009     
  Branches      27979    28096     +117     
============================================
+ Hits         115746   116022     +276     
- Misses        57662    58347     +685     
- Partials       8932     8980      +48     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 63.21% <40.00%> (-0.21%) ⬇️
java-21 63.22% <40.00%> (-0.23%) ⬇️
temurin 63.27% <40.00%> (-0.20%) ⬇️
unittests 63.27% <40.00%> (-0.20%) ⬇️
unittests1 56.25% <0.00%> (-0.07%) ⬇️
unittests2 33.63% <40.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xiangfu0 xiangfu0 merged commit 409f2ae into apache:master Oct 28, 2025
18 checks passed
@xiangfu0
Copy link
Contributor

Thanks for fixing this.

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.

3 participants