refactor: Use DeprecationWarning for deprecated request queue arguments

Author: vdusek

src/apify_client/clients/resource_clients/request_queue.py

@@ -1,8 +1,8 @@
 from __future__ import annotations
 
 import asyncio
-import logging
 import math
+import warnings
 from collections.abc import Iterable
 from queue import Queue
 from typing import TYPE_CHECKING, Any, TypedDict
@@ -23,8 +23,6 @@
 
     from apify_shared.consts import StorageGeneralAccess
 
-logger = logging.getLogger(__name__)
-
 _RQ_MAX_REQUESTS_PER_BATCH = 25
 _MAX_PAYLOAD_SIZE_BYTES = 9 * 1024 * 1024  # 9 MB
 _SAFETY_BUFFER_PERCENT = 0.01 / 100  # 0.01%
@@ -311,9 +309,17 @@ def batch_add_requests(
             Result containing lists of processed and unprocessed requests.
         """
         if max_unprocessed_requests_retries:
-            logger.warning('`max_unprocessed_requests_retries` is deprecated and not used anymore.')
+            warnings.warn(
+                '`max_unprocessed_requests_retries` is deprecated and not used anymore.',
+                DeprecationWarning,
+                stacklevel=2,
+            )
         if min_delay_between_unprocessed_requests_retries:
-            logger.warning('`min_delay_between_unprocessed_requests_retries` is deprecated and not used anymore.')
+            warnings.warn(
+                '`min_delay_between_unprocessed_requests_retries` is deprecated and not used anymore.',
+                DeprecationWarning,
+                stacklevel=2,
+            )
 
         if max_parallel != 1:
             raise NotImplementedError('max_parallel is only supported in async client')
@@ -393,8 +399,15 @@ def list_requests(
 
         Args:
             limit: How many requests to retrieve.
-            exclusive_start_id: All requests up to this one (including) are skipped from the result.
+            exclusive_start_id: Deprecated argument. Will be removed in next major release.
         """
+        if exclusive_start_id is not None:
+            warnings.warn(
+                '`exclusive_start_id` is deprecated for paginating requests.',
+                DeprecationWarning,
+                stacklevel=2,
+            )
+
         request_params = self._params(limit=limit, exclusiveStartId=exclusive_start_id, clientKey=self.client_key)
 
         response = self.http_client.call(
@@ -738,9 +751,17 @@ async def batch_add_requests(
             Result containing lists of processed and unprocessed requests.
         """
         if max_unprocessed_requests_retries:
-            logger.warning('`max_unprocessed_requests_retries` is deprecated and not used anymore.')
+            warnings.warn(
+                '`max_unprocessed_requests_retries` is deprecated and not used anymore.',
+                DeprecationWarning,
+                stacklevel=2,
+            )
         if min_delay_between_unprocessed_requests_retries:
-            logger.warning('`min_delay_between_unprocessed_requests_retries` is deprecated and not used anymore.')
+            warnings.warn(
+                '`min_delay_between_unprocessed_requests_retries` is deprecated and not used anymore.',
+                DeprecationWarning,
+                stacklevel=2,
+            )
 
         tasks = set[asyncio.Task]()
         queue: asyncio.Queue[Iterable[dict]] = asyncio.Queue()
@@ -821,8 +842,15 @@ async def list_requests(
 
         Args:
             limit: How many requests to retrieve.
-            exclusive_start_id: All requests up to this one (including) are skipped from the result.
+            exclusive_start_id: Deprecated argument. Will be removed in next major release.
         """
+        if exclusive_start_id is not None:
+            warnings.warn(
+                '`exclusive_start_id` is deprecated for paginating requests.',
+                DeprecationWarning,
+                stacklevel=2,
+            )
+
         request_params = self._params(limit=limit, exclusiveStartId=exclusive_start_id, clientKey=self.client_key)
 
         response = await self.http_client.call(