[SDK Automation] azure-sdk-for-java generate script build error

[JackTn]

After run azure-sdk-for-java Generate Script in sdk-automation.
It missing changelog content and version when input file is relatedTypeSpecProjectFolder or relatedReadmeMdFiles. The pipeline result you can see 1 2
The correct response refer to this

[raych1]

@weidongxu-microsoft @XiaofeiCao
both content and breakingChangeItems need to be added to report the breaking changes.

[weidongxu-microsoft]

@raych1

I did not get it. There is only 1 stable api-version in AzureFleet (computefleet as SDK)
https://github.com/Azure/azure-rest-api-specs/tree/main/specification/azurefleet/resource-manager/Microsoft.AzureFleet
And Java hadn't released it (it is scheduled today -- 10/22).

So, there is no breaking change to report at all -- there is no previous GA SDK, every SDK before is beta/preview

SDK only concerns about breaking change compared to a prior GA SDK (it be 2nd GA compares to 1st GA; or a next beta compares to 1st GA -- but you have to have a 1st GA SDK)

[JackTn]

@weidongxu-microsoft Apologies for any confusion caused. The issue at hand is that the breakingChangeItems are not being returned as expected. For a clearer understanding, please refer to the results shown here result. and here is correct response this.Additionally, it appears that the version has been returned accurately. Therefore, you may disregard it.

[weidongxu-microsoft]

@JackTn

I still didn't get you.

Result in azure-data-schemaregistry is because there was GA release 1.5.0 https://azure.github.io/azure-sdk/releases/latest/?search=azure-data-schemaregistry

No result for computefleet and datafactory is because there were no GA release in Java. (computefleet released 1.0.0 today -- that was after your pipeline) https://azure.github.io/azure-sdk/releases/latest/?search=azure-resourcemanager-datafactory

[weidongxu-microsoft]

And DO NOT COMPARE JAVA WITH GO. Breaking change depends on SDK state of each language, and they could be in different state.

Java SDK cares about whether some change could break a GA Java SDK. No concern about go SDK.

[weidongxu-microsoft]

You can reopen with a PR in specs, that it contains breaks to a released/GAed Java SDK, but it didn't have content in breaking changes section.

If you are unsure whether Java code change would be a break, check Revapi step in Java CI. https://dev.azure.com/azure-sdk/public/_build/results?buildId=4225865&view=logs&jobId=b70e5e73-bbb6-5567-0939-8415943fadb9&j=b70e5e73-bbb6-5567-0939-8415943fadb9&t=ba3cf43a-b2bf-50f9-f95f-93049ca44ad1

[JackTn]

Thansks, will check on this CI

[JackTn]

@raych1

I did not get it. There is only 1 stable api-version in AzureFleet (computefleet as SDK) https://github.com/Azure/azure-rest-api-specs/tree/main/specification/azurefleet/resource-manager/Microsoft.AzureFleet And Java hadn't released it (it is scheduled today -- 10/22).

So, there is no breaking change to report at all -- there is no previous GA SDK, every SDK before is beta/preview

SDK only concerns about breaking change compared to a prior GA SDK (it be 2nd GA compares to 1st GA; or a next beta compares to 1st GA -- but you have to have a 1st GA SDK)

As the AzureFleet is release, this pr has correct hasBreakingChange flag for JAVA. But why there are no breakingchange items return when the hasBreakingChange is true ? @weidongxu-microsoft

[weidongxu-microsoft]

@XiaofeiCao to take a look

@JackTn Which "breakingchange items" is it? This https://github.com/Azure/azure-rest-api-specs/blob/main/documentation/sdkautomation/GenerateOutputSchema.json#L73-L79 ? I guess Xiaofei didn't fill this one. It's added pretty late in Spec.

[XiaofeiCao]

@JackTn Seems this breakingChangeItems is added after we supported breaking change detection, see original issue #8477 (comment)

I'll add support for this property

[JackTn]

@JackTn Seems this breakingChangeItems is added after we supported breaking change detection, see original issue #8477 (comment)

I'll add support for this property

Thanks!