Skip to content

Update Java configuration for Log Analytics#3269

Merged
marstr merged 3 commits intoAzure:masterfrom
alexeldeib:java-loganalytics
Jun 25, 2018
Merged

Update Java configuration for Log Analytics#3269
marstr merged 3 commits intoAzure:masterfrom
alexeldeib:java-loganalytics

Conversation

@alexeldeib
Copy link
Contributor

@alexeldeib alexeldeib commented Jun 19, 2018 •
edited by azuresdkci
Loading

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.

PR information

  • The title of the PR is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
  • Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
  • If applicable, the PR references the bug/issue that it fixes.
  • Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

@azuresdkci azuresdkci requested a review from marstr June 19, 2018 22:55
alexeldeib
alexeldeib commented Jun 19, 2018
View reviewed changes
specification/operationalinsights/data-plane/readme.md
Please also specify `--azure-libraries-for-java-folder=<path to the root directory of your azure-libraries-for-java clone>`.

``` yaml $(java)
java:
Copy link
Contributor Author

@alexeldeib alexeldeib Jun 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no existing client that uses the old configuration.

alexeldeib
alexeldeib commented Jun 19, 2018
View reviewed changes
specification/operationalinsights/data-plane/readme.md
``` yaml
title: OperationalInsightsDataClient
description: Operational Insights Data Client
description: Log Analytics Data Plane Client
Copy link
Contributor Author

@alexeldeib alexeldeib Jun 19, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reflecting branding, which has been different for some time (keeping title due to existing packages dependent on it; will override on a per package basis for new releases).

@alexeldeib alexeldeib mentioned this pull request Jun 19, 2018
Merged
@alexeldeib
Copy link
Contributor Author

alexeldeib commented Jun 19, 2018

Related to Azure/azure-sdk-for-java#2113

@AutorestCI
Copy link

AutorestCI commented Jun 19, 2018 •
edited
Loading

Automation for azure-sdk-for-java

Nothing to generate for azure-sdk-for-java

@AutorestCI AutorestCI mentioned this pull request Jun 19, 2018
Merged
@AutorestCI
Copy link

AutorestCI commented Jun 19, 2018 •
edited
Loading

Automation for azure-sdk-for-python

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-python#2772

@AutorestCI
Copy link

AutorestCI commented Jun 19, 2018 •
edited
Loading

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

@AutorestCI AutorestCI mentioned this pull request Jun 19, 2018
Merged
@AutorestCI
Copy link

AutorestCI commented Jun 19, 2018 •
edited
Loading

Automation for azure-sdk-for-go

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-go#2126

@AutorestCI
Copy link

AutorestCI commented Jun 19, 2018 •
edited
Loading

Automation for azure-sdk-for-node

Nothing to generate for azure-sdk-for-node

@marstr marstr requested a review from jianghaolu June 20, 2018 23:27
@marstr
Copy link
Member

marstr commented Jun 20, 2018

I'd like @jianghaolu to validate the java configuration changes. Other than that, this looks good.

jianghaolu
jianghaolu reviewed Jun 20, 2018
View reviewed changes
specification/operationalinsights/data-plane/readme.md
## Java

These settings apply only when `--java` is specified on the command line.
Please also specify `--azure-libraries-for-java-folder=<path to the root directory of your azure-libraries-for-java clone>`.
Copy link
Contributor

@jianghaolu jianghaolu Jun 20, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you remove this?

Copy link
Contributor Author

@alexeldeib alexeldeib Jun 21, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry -- before the changes in generation, I saw this was fixed already in gulpfile. I'll undo this change.

@marstr
Copy link
Member

marstr commented Jun 21, 2018

@jianghaolu, does the update look good to you?

@jianghaolu
Copy link
Contributor

jianghaolu commented Jun 22, 2018

@marstr Yes! Please feel free to merge.

@marstr marstr merged commit 5a702f1 into Azure:master Jun 25, 2018
@alexeldeib alexeldeib mentioned this pull request Jul 2, 2018
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marstr marstr marstr approved these changes

+1 more reviewer

@jianghaolu jianghaolu jianghaolu left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@marstr marstr

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@alexeldeib @AutorestCI @marstr @jianghaolu