Batch sdk integration#2865
Merged
Merged
Conversation
Conflicts: azure-batch/src/main/java/com/microsoft/azure/batch/auth/BatchCredentialsInterceptor.java azure-batch/src/main/java/com/microsoft/azure/batch/protocol/BatchServiceClient.java azure-batch/src/main/java/com/microsoft/azure/batch/protocol/BatchServiceClientImpl.java azure-mgmt-datalake-analytics/src/test/java/com/microsoft/azure/management/datalake/analytics/DataLakeAnalyticsAccountOperationsTests.java azure-mgmt-datalake-analytics/src/test/java/com/microsoft/azure/management/datalake/analytics/DataLakeAnalyticsCatalogOperationsTests.java azure-mgmt-datalake-analytics/src/test/java/com/microsoft/azure/management/datalake/analytics/DataLakeAnalyticsJobOperationsTests.java azure-mgmt-datalake-analytics/src/test/java/com/microsoft/azure/management/datalake/analytics/DataLakeAnalyticsManagementTestBase.java azure-mgmt-datalake-store/src/test/java/com/microsoft/azure/management/datalake/store/DataLakeStoreFilesystemOperationsTests.java azure/pom.xml
…s some of the checkstyle and reference issue
Move non-top level classes out of implementation
* Update Java SDK to use latest rest api * update spacing * Update changelog * Update Note in changelog * Remove extra import * Infer Batch URL for BatchServiceClient from credentials * Update invalid javadoc comments * Update invalid javadoc comments * Update changelog for blobPrefix
Contributor
|
Can one of the admins verify this patch? |
g2vinay
requested review from
JonathanGiles and
lenala
and removed request for
JonathanGiles and
lenala
January 22, 2019 23:35
JonathanGiles
requested changes
Jan 23, 2019
xingwu1
reviewed
Feb 12, 2019
weshaggard
reviewed
Feb 13, 2019
Member
There was a problem hiding this comment.
Why do we need this travis.yml file?
Member
There was a problem hiding this comment.
I noted this also. My thinking was that we would clean up docs, remove redundant files, etc after we get the repo imported.
Member
There was a problem hiding this comment.
I'm fine with follow-up clean-up.
xingwu1
approved these changes
Feb 13, 2019
weshaggard
approved these changes
Feb 14, 2019
g2vinay
force-pushed
the
batch-sdk-integration
branch
from
February 14, 2019 20:34
36def3d to
dd7ace0
Compare
weshaggard
force-pushed
the
batch-sdk-integration
branch
from
February 14, 2019 21:46
66c2d3c to
18dbf7e
Compare
… Partial Checkstyle Fixes + Parallel Live test strategy implementation in Dev Ops + Batch SDK release update.
g2vinay
force-pushed
the
batch-sdk-integration
branch
from
February 14, 2019 23:12
18dbf7e to
b079a7f
Compare
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
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.
Includes changes of
Azure Batch base code migration.
Enabled Live Tests.