arcus-azure / arcus-azure/arcus.testing
🕵️ Investigate in hard-coded `100` in `TemporaryDataFlowDebugSession`
Open
area:integration
management
- Dominant language
- C#
- Stars
- 9
- Forks
- 6
- PR merge metrics
- No merged PRs in 30d
Description
We should investigate in this line in the `TemporaryDataFlowDebugSession`:
```csharp
debug.DebugSettings.SourceSettings.Add(new DataFlowSourceSetting { SourceName = source.Name, RowLimit = 100 });
```
Is the hard-coded `100` correct? And should this be the same as the `options.MaxRows`?
Contributor guide
Research direction
Start at TemporaryDataFlowDebugSession and inspect the DataFlowSourceSetting construction alongside options.MaxRows. Determine whether RowLimit should remain 100 or match options.MaxRows, then document the decision and verify the resulting behavior with the relevant project tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- azure, csharp
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100