arcus-azure / arcus-azure/arcus.testing

🕵️ Investigate in hard-coded `100` in `TemporaryDataFlowDebugSession`

Open
#229 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.