TheNamesRai commented on code in PR #2307:
URL: https://github.com/apache/phoenix/pull/2307#discussion_r2468703078
##########
phoenix-core-server/src/main/java/org/apache/phoenix/util/ServerUtil.java:
##########
@@ -291,4 +291,33 @@ public static <T> Throwable
getExceptionFromFailedFuture(Future<T> f) {
}
return t;
}
+
+ /**
+ * Derives a safe row key for empty result sets based on scan or region
boundaries. Used when
+ * constructing KeyValues for aggregate results or OFFSET responses when no
actual data rows were
+ * scanned.
+ * @param scan The scan being executed
+ * @param region The region being scanned
+ * @return A valid row key derived from scan or region boundaries
+ */
+ public static byte[] deriveRowKeyFromScanOrRegionBoundaries(Scan scan,
Region region) {
Review Comment:
Naming convention for this method is similar to
`getScanStartRowKeyFromScanOrRegionBoundaries` method in the ServerUtil.java
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
