Skip to content
This repository was archived by the owner on Mar 31, 2025. It is now read-only.

Commit a926f7e

Browse files
committed
add warning for pageLimit>100
1 parent b04ebad commit a926f7e

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

src/main/java/com/sybit/airtable/Table.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,6 +163,7 @@ public List<T> select(final Query query) throws AirtableException {
163163
}
164164
if (query.getPageSize() != null) {
165165
if (query.getPageSize() > 100) {
166+
LOG.warn("pageSize is limited to max 100 but was " + query.getPageSize());
166167
request.queryString("pageSize", 100);
167168
} else {
168169
request.queryString("pageSize", query.getPageSize());
@@ -187,7 +188,7 @@ public List<T> select(final Query query) throws AirtableException {
187188
}
188189

189190
int code = response.getStatus();
190-
List<T> list = null;
191+
List<T> list;
191192
if (200 == code) {
192193
list = getList(response);
193194

@@ -198,6 +199,7 @@ public List<T> select(final Query query) throws AirtableException {
198199
}
199200
} else {
200201
HttpResponseExceptionHandler.onResponse(response);
202+
list = null;
201203
}
202204

203205
return list;

0 commit comments

Comments
 (0)