Opened 23 months ago

Last modified 7 months ago

#21365 assigned task

Investigate whether descriptor parsing is guaranteed to be thread-safe

Reported by: iwakeh Owned by: metrics-team
Priority: Medium Milestone:
Component: Metrics/Library Version:
Severity: Normal Keywords: metrics-2018
Cc: metrics-team Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Partially discovered in #20440 and should be thoroughly verified.
Class affected BlockingIteratorImpl

Child Tickets

Change History (13)

comment:1 Changed 23 months ago by karsten

I'm unclear what makes you think that metrics-lib is not thread-safe. I mean, this could be the case. But the issue in #20440 did not indicate problems with thread-safety, did it? Can you give more details?

comment:2 Changed 23 months ago by iwakeh

My investigations back than for #20440 made me aware of this.

I'll device a test for (re)producing the concurrency issues in BlockingIteratorImpl.

comment:3 Changed 21 months ago by karsten

Status: newneeds_information
Summary: metrics-lib might not be thread savemetrics-lib might not be thread-safe

Setting to needs_information until there's a way to reproduce this issue.

comment:4 Changed 19 months ago by iwakeh

Milestone: metrics-lib 1.8.0
Type: defecttask

comment:5 Changed 19 months ago by karsten

Milestone: metrics-lib 1.8.0

This is potentially a hidden major bug, but it seems like it won't be relevant until we get to #21751. And we likely won't get to that before 2.1.0 for which we don't have a milestone yet. Unassigning from milestones until we make new plans. (The same goes for #21751.)

comment:6 Changed 18 months ago by karsten

Note: Also look into #22678 when working on this.

comment:7 Changed 15 months ago by karsten

Summary: metrics-lib might not be thread-safeInvestigate whether descriptor parsing is guaranteed to be thread-safe

Tweak summary a bit.

comment:8 Changed 15 months ago by karsten

Keywords: metrics-2018 added

comment:9 Changed 15 months ago by karsten

Keywords: metrics-2017 added; metrics-2018 removed

comment:10 Changed 12 months ago by iwakeh

Keywords: metrics-2018 added; metrics-2017 removed

Will be completed in 2018.

comment:11 Changed 9 months ago by iwakeh

Owner: changed from metrics-team to iwakeh
Status: needs_informationaccepted

Waiting on the solution to #20395.

comment:12 Changed 9 months ago by irl

Cc: metrics-team added

Adding metrics-team to cc

comment:13 Changed 7 months ago by iwakeh

Owner: changed from iwakeh to metrics-team
Status: acceptedassigned

Won't be starting work on this next week. -> re-assigning to metrics-team.

Note: See TracTickets for help on using tickets.