This repository was archived by the owner on Aug 26, 2025. It is now read-only.
[AutoPR compute/resource-manager/v2017_03_30] Redeploy#427
Closed
Conversation
* remove java code isImple check but move to suppression and add only check for public class for external Dependency check * fixes * Update eng/code-quality-reports/src/main/resources/checkstyle/checkstyle-suppressions.xml suggest changes on comments Co-Authored-By: Connie Yau <conniey@microsoft.com>
* Move Storage to official swagger files There are no (meaningful) code changes except the attribute in BlobHierarchyListSegment that I can't trace to the swagger * Upgrade blobs swagger to 2019-02-02 Note that additional work will have to be done to patch our higher level code to pass new parameters to existing APIs * Fixes to Swagger generation and updating to use new protocol layer * Fixed appendBlock content-type, added SpotBug exclusions for autogen code * Update README.md * Fixed incorrect exclude
* Migrate queue to spock test framework * Remove all Junit tests and dependencies
…literals usage (#4994) * Adding javaDoc formating rule * Adding fixes for javadoc formatting
* Add staging profile.
…ion mark, if so remove it (#5024)
* Created StorageException for Files and Queues, moved it into models for Blobs, made it a POJO * Using getX naming and cleaned up unit test imports * Fixed additional imports * Fixed import and ignored issue with enum serialization
* Smoke Test for Key Vault, Storage and Identity SDK * Create EventHubs.java * Update EventHubs.java * Update KeyVaultSecrets.java * Update App.java * Create CosmosDB files * CosmosDB finallized * Update EventHubs.java * Handling pre-cleaning the resource * Update App.java * Concurrency support * Update pom.xml * Review feedback * Reactive async in Cosmos * Use of Logger * Env Variables names changed * Create README.md * Use of logger in error message * Location changed to /eng/ * Create smoke-test.yml * Update smoke-test.yml * Update smoke-test.yml * Update smoke-test.yml * Update README.md * Website in pom.xml * MIT headers * static final named correctly * Avoiding import.* * nit: extra blank lines and logger
KV Track 1 OkHttp Bug fix.
…kstyle errors (#5032) * remove java code isImple check but move to suppression and add only check for public class for external Dependency check * refactor and fixes errors * add break
…sues are fixed (#5005) * Adding rule for enforcing final field usage
* Update Storage Files swagger to 2019-02-02 Also adding a transform to automatically add the serialization attribute back on hierarchical blobs when regenerating * Updating to use new service interface * Updating Swagger transforms to fix deserialization issue and inappropriate field issue * Disable failing test to fix later * Update transform header to explain its use
* Remove blocking calls from ChangeFeedProcessor implementation and other various fixes. * Address PR feedback. * static logger * refactoring self->this remove Thread.sleep() call from some of the loops * Remove additional calls to Thread.sleep(). * Remove redundant calls to Exceptions.propagate() * Limit the feeds to one in order to avoid potential concurency issues where earlier feeds can be processed slower than newer feeds and wrong checkpoint is saved in the lease document * PR related refactoring. * PR related fix. * PR related changes. * Add volatile to class members accessed from reactor code. * Add volatile to class members accessed from reactor code. * Add volatile to class members accessed from reactor code. * Replace usage of ExecutorService with Reactor's Scheduler (per convesation with Chris). Code style fixes in anticipation of liner rules that will be eventually enabled for the SDK. * The exception that results from the the child threads should be marked as volatile since it can be accessed via parallel running threads.
This port brings v3 into alignment with v2.6.1 of the Cosmos SDK for Java. Except for RntbdTransportClient.java which has some adjustments for Reactor the implementations are now equivalent.
(1) the implementation for Rule 9: Thrown exception use ClientLogger To throw an exception, Must throw it through a 'logger.logExceptionAsError', rather than by directly calling 'throw exception'. Skip check if throw exception from Static method Static class Constructor (2) code fixes based on the new rule: ThrowFromClientLoggerCheck.java (3) a minor fixes for NoImplInPublicAPI.java checker: Removes track 2 and implementation check in Java Code instead of suppressing the rule (4) fixes for ServiceClientBuilder checker for Storage service.
* Fixed JavadocPackage supression, removed any newlines between regex ORs * Cleaned up merge changes
… artifacts (#5433) * Increment the current versions of com.microsoft.azure:azure-keyvault* artifacts * update spotbugs-aggregate-report for keyvault version
…iction and azure-cognitiveservices-customvision-training to 1.1.0-preview.3 (#5437)
…5181) * Very early stages of introducing module-info.java files. Code does not compile and tests do not pass. * Making more modules progress * A bunch more work on modules. Now it is failing on my machine with the groovy-eclipse-compiler not recognizing '--module-version' option. * Skip groovy tests and fix a few other JDK 9 related issues * Add module-info.java * Update azure-core module-info.java to export to storage modules * revert storage blob pom file * Spock and groovy tests for storage * Fix test failures * Fix storage tests * Updates for java8 * Fix groovy tests after rebase * Add module-info.java to new packages * Add module-info.java to core management * Fix checkstyle and spotbugs * Update module-info after rebase * Fix RestProxyXMLTests * Address PR comments * Remove empty packages from module info * Update storage imports for new package * Change profile name from java9plus to java-lts to match CI pipeline names * Remove core test package from javadoc reports
* Code samples for common credentials * Cleaning up linting issues
…#5452) * Adding EventHubReactorSession that implements EventHubSession. * Use EventHubReactorSession in Connection. * Fix NullPointerExceptions in SessionHandler and the occasional one in super.onUnhandled.
…#5432) * Cosmos version changes * cosmos current version update * update the cosmos ci.yml file * remove smoke-tests change from PR
* Changed default retry timeout and extended test jitter bounds to fix transient tests * Extending retry bounds more
Removed extra file
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Created to sync Azure/azure-rest-api-specs#7294