Copilot commented on code in PR #16978:
URL: https://github.com/apache/pinot/pull/16978#discussion_r2415076040
##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/ConsistentDataPushUtils.java:
##########
@@ -236,8 +236,10 @@ public static Map<URI, List<String>>
getSegmentsToReplace(SegmentGenerationJobSp
List<String> offlineSegments;
try {
controllerURI = new URI(pinotClusterSpec.getControllerURI());
+ AuthProvider authProvider =
AuthProviderUtils.makeAuthProvider(spec.getAuthToken());
Review Comment:
The indentation is inconsistent. The line should be aligned with the
surrounding code at the same level as line 238.
```suggestion
AuthProvider authProvider =
AuthProviderUtils.makeAuthProvider(spec.getAuthToken());
```
##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/ConsistentDataPushUtils.java:
##########
@@ -236,8 +236,10 @@ public static Map<URI, List<String>>
getSegmentsToReplace(SegmentGenerationJobSp
List<String> offlineSegments;
try {
controllerURI = new URI(pinotClusterSpec.getControllerURI());
+ AuthProvider authProvider =
AuthProviderUtils.makeAuthProvider(spec.getAuthToken());
Map<String, List<String>> segments =
- FILE_UPLOAD_DOWNLOAD_CLIENT.getSegments(controllerURI,
rawTableName, TableType.OFFLINE, true);
+ FILE_UPLOAD_DOWNLOAD_CLIENT.getSegments(controllerURI,
rawTableName, TableType.OFFLINE,
+ true, authProvider);
Review Comment:
The indentation is inconsistent. Line 242 should be aligned properly with
the method parameters, and the extra spaces should be normalized to match the
project's formatting style.
```suggestion
true, authProvider);
```
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
