facultyai / facultyai/faculty

list_jobs limit argument needs clarification or more suitable default value

オープン
#169 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Python
スター
16
フォーク
5
PR マージ指標
30日以内にマージされた PR はありません

説明

In `JobClient`, the `list_jobs` function has a `limit` argument, which sets the number of jobs returned in a single request. The default value is is `None` in the library, which means "use the default limit from the backend service". That default value is (I believe) `10`.

https://github.com/facultyai/faculty/blob/master/faculty/clients/job.py#L314

Had a request to use that upstream default value in the library to make the effect of different values clearer. Otherwise interpreting the behaviour of `None` is difficult (for example will there be paging in that case or all records are returned?)

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。