diff --git a/library/core/src/main/java/com/google/android/exoplayer2/upstream/cache/CacheKeyFactory.java b/library/core/src/main/java/com/google/android/exoplayer2/upstream/cache/CacheKeyFactory.java
index 2236b5f9cc4..69e9b73fdde 100644
--- a/library/core/src/main/java/com/google/android/exoplayer2/upstream/cache/CacheKeyFactory.java
+++ b/library/core/src/main/java/com/google/android/exoplayer2/upstream/cache/CacheKeyFactory.java
@@ -25,10 +25,15 @@ public interface CacheKeyFactory {
       (dataSpec) -> dataSpec.key != null ? dataSpec.key : dataSpec.uri.toString();
 
   /**
-   * Returns a cache key for the given {@link DataSpec}.
+   * Returns the cache key of the resource containing the data defined by a {@link DataSpec}.
    *
-   * @param dataSpec The data being cached.
-   * @return The cache key.
+   * <p>Note that since the returned cache key corresponds to the whole resource, implementations
+   * must not return different cache keys for {@link DataSpec DataSpecs} that define different
+   * ranges of the same resource. As a result, implementations should not use fields such as {@link
+   * DataSpec#position} and {@link DataSpec#length}.
+   *
+   * @param dataSpec The {@link DataSpec}.
+   * @return The cache key of the resource.
    */
   String buildCacheKey(DataSpec dataSpec);
 }