Skip to content

Commit 47d2f42

Browse files
prmoore77claude
andcommitted
fix: Add missing Javadoc comments to fix checkstyle violations
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 9e4480f commit 47d2f42

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

flight/flight-sql-jdbc-core/src/main/java/org/apache/arrow/driver/jdbc/client/oauth/OAuthConfiguration.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -239,6 +239,13 @@ public Builder tokenUri(@Nullable String tokenUri) throws SQLException {
239239
return this;
240240
}
241241

242+
/**
243+
* Sets the OAuth authorization endpoint URL.
244+
*
245+
* @param authorizationUrl the authorization endpoint URL
246+
* @return this builder
247+
* @throws SQLException if the URL is invalid
248+
*/
242249
public Builder authorizationUrl(@Nullable String authorizationUrl) throws SQLException {
243250
if (authorizationUrl == null || authorizationUrl.isEmpty()) {
244251
this.authorizationUri = null;
@@ -252,6 +259,7 @@ public Builder authorizationUrl(@Nullable String authorizationUrl) throws SQLExc
252259
return this;
253260
}
254261

262+
/** Sets the OIDC issuer URL for endpoint auto-discovery. */
255263
public Builder oidcIssuer(@Nullable String oidcIssuer) {
256264
this.oidcIssuer = oidcIssuer;
257265
return this;

flight/flight-sql-jdbc-core/src/main/java/org/apache/arrow/driver/jdbc/client/oauth/PkceGenerator.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,11 @@ public PkceGenerator() {
4040
this(DEFAULT_VERIFIER_LENGTH);
4141
}
4242

43+
/**
44+
* Creates a new PkceGenerator with a custom verifier length.
45+
*
46+
* @param verifierLength the length of the code verifier (43-128 characters)
47+
*/
4348
public PkceGenerator(int verifierLength) {
4449
if (verifierLength < 43 || verifierLength > 128) {
4550
throw new IllegalArgumentException("Verifier length must be between 43 and 128 characters");

0 commit comments

Comments
 (0)