-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
feat: adding hadoopfs sample #2949
Conversation
This might be blocked by: #294 |
Fixes #2928 > It's a good idea to open an issue first for discussion. - [x] I have followed [Sample Format Guide](https://github.com/GoogleCloudPlatform/java-docs-samples/blob/master/SAMPLE_FORMAT.md) - [x] `pom.xml` parent set to latest `shared-configuration` - [ ] Appropriate changes to README are included in PR - [ ] API's need to be enabled to test (tell us) - [ ] Environment Variables need to be set (ask us to set them) - [ ] Tests pass (`mvn -P lint clean verify`) * (Note- `Checkstyle` passing is required; `Spotbugs`, `ErrorProne`, `PMD`, etc. `ERROR`'s are advisory only) - [x] Please **merge** this PR for me once it is approved.
* fix: talent solution v4 samples tests * deleted unnecessary text file * updated exp date * fixed the lint * updated code according to new checkstyle * removed setExpireDate mthd * added helpful link to job exp
* chore: added comment on try/catch block * started breaking down detect file * nits * formatted new files
Modifies the deployment methods to the new source deploy functionality for those applications that do not have a dependency on the appengine-simple-jetty-main artifact
OperationFuture<Job, JobMetadata> submitJobAsOperationAsyncRequest = | ||
jobControllerClient.submitJobAsOperationAsync(projectId, region, job); | ||
|
||
Job response = submitJobAsOperationAsyncRequest.get(); |
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.
could this Job fail and throw exceptions that should be caught and handled?
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.
If the job fails, it'l throws an execution exception. I have that captured below.
System.out.println(String.format("Job \"%s\" finished: %s", | ||
response.getReference().getJobId(), | ||
new String(blob.getContent()))); | ||
|
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.
if job submission succeed, maybe we could just print out "Job successfully submitted" and check this in the IT.
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.
I can remove the job id, but the content of the output is vital for this use case.
…uth2-http to v0.21.0 (#3244)
…abind to v2.11.1 (#3250)
…telligence to v1.4.0 (#3248)
…#2986) * refactor: use new verify token functionality from google-auth-library * fix: restore check for subject and email * docs: clarify comment about subject/email claims * fix: print verification exception message to stdout
* chore(deps): update dependency com.google.cloud:google-cloud-kms to v1.39.0 * add protobuf-java-util as a direct dep Co-authored-by: Renovate Bot <[email protected]>
…to v1.111.0 (#3253) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [com.google.cloud:google-cloud-storage](https://togithub.com/googleapis/java-storage) | minor | `1.110.0` -> `1.111.0` | --- ### Release Notes <details> <summary>googleapis/java-storage</summary> ### [`v1.111.0`](https://togithub.com/googleapis/java-storage/blob/master/CHANGELOG.md#​11110-httpswwwgithubcomgoogleapisjava-storagecomparev11100v11110-2020-06-25) [Compare Source](https://togithub.com/googleapis/java-storage/compare/v1.110.0...v1.111.0) ##### Features - add storage.upload(path) ([#​269](https://www.github.com/googleapis/java-storage/issues/269)) ([9457f3a](https://www.github.com/googleapis/java-storage/commit/9457f3a76ff18552adc5f9c82f62ab8f3c207d31)) - Add support to disable logging from bucket ([#​390](https://www.github.com/googleapis/java-storage/issues/390)) ([be72027](https://www.github.com/googleapis/java-storage/commit/be72027b1587b9b0a3e9e65e7a2231bdb2ae521f)) - expose all the methods of notification ([#​141](https://www.github.com/googleapis/java-storage/issues/141)) ([8dfc0cb](https://www.github.com/googleapis/java-storage/commit/8dfc0cbf8294a7fc426948e22e5c2182da97b630)) ##### Reverts - Revert "feat: expose all the methods of notification ([#​141](https://togithub.com/googleapis/java-storage/issues/141))" ([#​393](https://togithub.com/googleapis/java-storage/issues/393)) ([3e02b9c](https://www.github.com/googleapis/java-storage/commit/3e02b9c4ee1ce0fb785d15b04bd36754e31831a0)), closes [#​141](https://www.github.com/googleapis/java-storage/issues/141) [#​393](https://www.github.com/googleapis/java-storage/issues/393) </details> --- ### Renovate configuration :date: **Schedule**: At any time (no schedule defined). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Never, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#GoogleCloudPlatform/java-docs-samples).
…nager to v1.1.0 (#3237)
…center to v1.1.0 (#3245)
#3260) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [org.scala-lang:scala-library](https://www.scala-lang.org/) ([source](https://togithub.com/scala/scala)) | patch | `2.13.2` -> `2.13.3` | --- ### Release Notes <details> <summary>scala/scala</summary> ### [`v2.13.3`](https://togithub.com/scala/scala/releases/v2.13.3) [Compare Source](https://togithub.com/scala/scala/compare/v2.13.2...v2.13.3) Scala 2.13.3 is primarily a bugfix release. It also includes: - improvements to warnings and linting - experimental `-Xasync` support For more detail, read on. ##### Behavior changes - `Symbol#toString` is now `Symbol(foo)` instead of the deprecated single-quote form `'foo` ([#​8933](https://togithub.com/scala/scala/pull/8933)) ##### Bugfixes - Fix 2.13-only bug in Java collection converters that caused some operations to perform an extra pass ([#​9058](https://togithub.com/scala/scala/pull/9058)) - Fix 2.13.2 performance regression in `Vector`: restore special cases for small operands in `appendedAll` and `prependedAll` ([#​9036](https://togithub.com/scala/scala/pull/9036)) - Increase laziness of `#::` for `LazyList` ([#​8985](https://togithub.com/scala/scala/pull/8985)) - Append trailing `/` when processing unicode escapes ([#​8942](https://togithub.com/scala/scala/pull/8942)) - Respect `@uncheckedVariance` in higher-kinded types (fixing 2.13.2 regression) ([#​8938](https://togithub.com/scala/scala/pull/8938)) ##### Warnings and linting - Deprecate auto-application of (non-Java-defined) methods with a single empty parameter list ([#​8833](https://togithub.com/scala/scala/pull/8833)) - The PR has instructions for suppressing the warning if it is unwanted - Warn by default on mismatch of presence/absence of an empty parameter list when overriding ([#​8846](https://togithub.com/scala/scala/pull/8846)) - Discourage multi-argument infix syntax: lint applications (`x op (a, b)`), also lint operator-name definitions ([#​8951](https://togithub.com/scala/scala/pull/8951)) - Fix `@nowarn` to use correct semantics for `&` ([#​9032](https://togithub.com/scala/scala/pull/9032)) - Make `-Wunused:imports` work again even when `-Ymacro-annotations` is enabled ([#​8962](https://togithub.com/scala/scala/pull/8962)) - Replace `-Wself-implicit` with `-Xlint:implicit-recursion` ([#​9019](https://togithub.com/scala/scala/pull/9019)) - Under `-Xsource:3`, disallow auto-eta-expansion of SAMs ([#​9049](https://togithub.com/scala/scala/pull/9049)) ##### Experimental `-Xasync` This successor to [scala-async](https://togithub.com/scala/scala-async) allows usage with other effect systems besides `scala.concurrrent.Future`. - Compiler support for scala-async; enable with `-Xasync` ([#​8816](https://togithub.com/scala/scala/pull/8816)) We will publish a blog post with more detail on this work by [@​retronym](https://togithub.com/retronym), building on his earlier collaboration with [@​phaller](https://togithub.com/phaller). In the meantime, see the PR description. This feature will also be included in the 2.12.12 release. ##### Other changes For complete 2.13.3 change lists, see [all merged PRs](https://togithub.com/scala/scala/pulls?q=is%3Amerged%20milestone%3A2.13.3) and [all closed bugs](https://togithub.com/scala/bug/issues?utf8=%E2%9C%93&q=is%3Aclosed+milestone%3A2.13.3). Some changes that will ship in 2.12.12 are also included in this release, most notably: - Annotation parsing & `@Deprecated` ([#​8781](https://togithub.com/scala/scala/pull/8781)) - Fix Scaladoc tool on JDK 11 with `-release 8`: exclude sig files in `Symbol#sourceFile` ([#​8849](https://togithub.com/scala/scala/pull/8849)) ##### Compatibility As usual for our minor releases, Scala 2.13.3 is binary-compatible with the whole Scala 2.13 series. Upgrading from 2.12? Enable `-Xmigration` during upgrade to request migration advice from the compiler. ##### Contributors A big thank you to everyone who's helped improve Scala by reporting bugs, improving our documentation, spreading kindness in discussions around Scala, and submitting and reviewing pull requests! You are all magnificent. This release was brought to you by 28 contributors, according to `git shortlog -sn --no-merges HEAD ^v2.13.2 ^2.12.x`. Thank you A. P. Marki, Jason Zaugg, Seth Tisue, Dale Wijnand, Lukas Rytz, Georgi Krastev, David Barri, Eugene Yokota, Diego E. Alonso Blas, Akhtiam Sakaev, Glenn Liwanag, changvvb, Evgeny Ganchurin, Mike Skells, Martijn Hoekstra, yudedako, Anatolii Kmetiuk, Gilles Peiffer, JyotiSachdeva.ext, Karol Chmist, Kenji Yoshida, Lorenzo Costanzia di Costigliole, NthPortal, Steven Barnes, Sébastien Doeraene, Travis Brown, counter2015, nogurenn. Thanks to [Lightbend](https://www.lightbend.com/scala) for their continued sponsorship of the Scala core team’s efforts. Lightbend offers [commercial support](https://www.lightbend.com/lightbend-platform-subscription) for Scala. ##### Scala 2.13 notes The [release notes for Scala 2.13.0](https://togithub.com/scala/scala/releases/v2.13.0) have important information applicable to the whole 2.13 series. ##### Obtaining Scala Scala releases are available through a variety of channels, including (but not limited to): - Bump the `scalaVersion` setting in your sbt-based project - Download a distribution from [scala-lang.org](http://scala-lang.org/download/2.13.3.html) - Obtain JARs via [Maven Central](http://search.maven.org/#search%7Cga%7C1%7Cg%3A%22org.scala-lang%22%20AND%20v%3A%222.13.3%22) </details> --- ### Renovate configuration :date: **Schedule**: At any time (no schedule defined). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Never, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#GoogleCloudPlatform/java-docs-samples).
…3251) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [com.google.cloud:libraries-bom](https://togithub.com/GoogleCloudPlatform/cloud-opensource-java) | major | `7.0.2` -> `8.0.0` | | [com.google.cloud:libraries-bom](https://togithub.com/GoogleCloudPlatform/cloud-opensource-java) | major | `5.2.0` -> `8.0.0` | | [com.google.cloud:libraries-bom](https://togithub.com/GoogleCloudPlatform/cloud-opensource-java) | major | `5.4.0` -> `8.0.0` | --- ### Renovate configuration :date: **Schedule**: At any time (no schedule defined). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Never, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#GoogleCloudPlatform/java-docs-samples).
That might be too many files to review - by a long shot. |
Why don't you start a new PR? |
Adding a sample + test for submitting a HadoopFS command using the Java client library. Adding a sample + test for creating a Dataproc cluster with autoscaling configured. Follow-up to #2949
Adding a sample + test for submitting a HadoopFS command using the Java client library. Adding a sample + test for creating a Dataproc cluster with autoscaling configured. Follow-up to #2949
This sample submits a Hadoop FS command to a Dataproc cluster (-ls, -du, etc.)