chapel-lang/chapel

Test that all IO routines on readers/writers/files work from remote locales

Open

#24,426 opened on Feb 15, 2024

View on GitHub
 (2 comments) (2 reactions) (0 assignees)Chapel (446 forks)auto 404
area: Libraries / Moduleseasy / straightforwardgood first issue

Repository metrics

Stars
 (2,010 stars)
PR merge metrics
 (PR metrics pending)

Description

PR #24422 fixed a bug in which our seek() routines did not contain an on-clause before calling into the C IO routines, resulting in a segfault. This issue captures a desire to test all methods on fileReaders/fileWriters/files from a remote locale to ensure that other such bugs aren't lurking.

As an example of how such tests might look, here's a pattern that tests .seek() on a fileReader:

var infile = open("out.txt", ioMode.r);
on Locales[numLocales-1] {
  var myReader = infile.reader(locking=false);
  myReader.seek(2..#2);
  var i = myReader.read(int);
  writeln(i);
}

Contributor guide

Research direction
Write tests for all IO routines (e.g., read, write, seek, etc.) on fileReaders, fileWriters, and files from a remote locale, following the example pattern using `on Locales[numLocales 1]`.
Tech stack
None
Domain
backend
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitChapel
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70