-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Added @JsonIgnore to getLogger in JsonSerializable #5951
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
Added @JsonIgnore to getLogger in JsonSerializable #5951
Conversation
|
/azp run java - cosmos - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
| <parent> | ||
| <groupId>com.microsoft.azure</groupId> | ||
| <artifactId>azure-cosmos-parent</artifactId> | ||
| <version>3.4.0</version> <!-- {x-version-update;com.microsoft.azure:azure-cosmos-parent;current} --> |
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.
Just to double check - 3.4.0 was just a snapshot right?
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.
Not anymore.
The practice has been changed to update the versions to next to be released version.
| private final static ObjectMapper OBJECT_MAPPER = Utils.getSimpleObjectMapper(); | ||
| private ObjectMapper om; | ||
| transient ObjectNode propertyBag = null; | ||
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.
Please refrain from making unrelated formatting changes, you can configure intellij to stop doing this automatically
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 really wanted to, but couldn't stop it. :(
You can show me a demo sometime :)
|
CC: @weshaggard Last but not least, when versioning files please take a look at the Versions and versioning section of https://github.com/Azure/azure-sdk-for-java/blob/master/CONTRIBUTING.md. The versions of Cosmos versions should be updated in version_data.txt and then the update_versions.py script should be run which will update the pom and readme files. There's a secondary PR for this to update the version in java files as well which is going to run into merge issues of these version changes. |
@JimSuplizio, this wasn't a hotfix, but more like a small bug fix (that was needed in spring-data-cosmosdb SDK) that wasn't related to any previous changes. Going forward, I will make sure to create a hotfix branch, and follow the hotfix pattern for any bug fixes, or hotfixes releases. Regarding tags, I couldn't find any tags under azure-sdk-for-java that were related to cosmos SDK previous releases. Since cosmos SDK and azure-sdk-for-java follow different release cycles, how can we create tags for previous versions that got released, and for this release as well? Also, do we need to update version_data.txt every time we release a new cosmos SDK version? For now, I can create a PR that updates the version_data.txt file. Will that be okay? |
The github UI doesn't do well with listing all the tags I suggest using the command line |
port from: Azure#5951
Uh oh!
There was an error while loading. Please reload this page.