-
Notifications
You must be signed in to change notification settings - Fork 68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
samples: Add StorageControl Quickstart Sample #2479
Conversation
Here is the summary of changes. You are about to add 1 region tag.
This comment is generated by snippet-bot.
|
Warning: This pull request is touching the following templated files:
|
0fd293b
to
950e13d
Compare
0197e9a
to
d39feb2
Compare
d39feb2
to
9b206f5
Compare
samples/snippets/src/main/java/com/example/storage/QuickstartStorageControlSample.java
Outdated
Show resolved
Hide resolved
public static void main(String... args) throws Exception { | ||
String bucketName = args[0]; // "your-bucket-name"; | ||
StorageControlClient storageControlClient = StorageControlClient.create(); | ||
GetStorageLayoutRequest request = GetStorageLayoutRequest.newBuilder() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May help to clarify that _
represents global bucket namespace;
6cfb588
to
aff6140
Compare
@@ -28,9 +28,6 @@ | |||
public class QuickstartStorageControlSample { | |||
public static void main(String... args) throws Exception { | |||
String bucketName = args[0]; // "your-bucket-name"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linter will probably chime in but remove the imports for storage client as well
88b5691
to
20d795f
Compare
Fixes #2477