Skip to content

Conversation

@hubgeter
Copy link
Contributor

@hubgeter hubgeter commented Apr 28, 2025

bp #50161

@hubgeter hubgeter requested a review from yiguolei as a code owner April 28, 2025 06:37
@Thearas
Copy link
Contributor

Thearas commented Apr 28, 2025

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@hubgeter
Copy link
Contributor Author

run buildall

1 similar comment
@hubgeter
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

BE UT Coverage Report

Increment line coverage 41.67% (5/12) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 38.84% (10269/26441)
Line Coverage 29.90% (85302/285290)
Region Coverage 28.55% (43959/153977)
Branch Coverage 25.28% (22486/88936)

@morningman morningman force-pushed the pick_21_fix_par_renaem_tb branch from 11e7416 to 6b4344d Compare April 29, 2025 04:54
@morningman
Copy link
Contributor

run buildall

@hello-stephen
Copy link
Contributor

BE UT Coverage Report

Increment line coverage 41.67% (5/12) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 38.86% (10275/26441)
Line Coverage 29.92% (85359/285297)
Region Coverage 28.57% (43988/153984)
Branch Coverage 25.31% (22507/88942)

hubgeter added 2 commits May 6, 2025 10:23
…o read files, there will be multiple threads modify same object (apache#50161)

Related PR: apache#38432

Problem Summary:
in pr apache#38432 , if parquet reader use index to reade file and file column
name not eq table column name, reader will modify
_colname_to_value_range . However, this object is held by multiple vfile
scanners, and multi-threaded modification of this object will cause be
core.
@morningman morningman force-pushed the pick_21_fix_par_renaem_tb branch from 6b4344d to 474180c Compare May 6, 2025 02:23
@morningman
Copy link
Contributor

run buildall

@hello-stephen
Copy link
Contributor

BE UT Coverage Report

Increment line coverage 41.67% (5/12) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 38.87% (10278/26442)
Line Coverage 29.92% (85381/285366)
Region Coverage 28.58% (44023/154040)
Branch Coverage 25.31% (22523/88982)

@yiguolei yiguolei merged commit 5501e13 into apache:branch-2.1 May 8, 2025
19 of 20 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.

6 participants