search

Azure / azure-sdk-for-java

This repository is for active development of the Azure SDK for Java. For consumers of the SDK we recommend visiting our public developer docs at https://docs.microsoft.com/java/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-java.
MIT License
2.25k stars 1.93k forks source link

[QUERY] Do we need to show "AZURE_COSMOS_DISABLE_NON_STREAMING_ORDER_BY" in the logs #40709

Closed yanfang-ma closed 2 days ago

yanfang-ma commented 2 weeks ago

Query/Question

We can see below logs when using azure-cosmos 4.61.1 Jun 19, 2024 3:37:50 PM com.azure.cosmos.implementation.Configs getAzureCosmosNonStreamingOrderByDisabled INFO: AZURE_COSMOS_DISABLE_NON_STREAMING_ORDER_BY property is: null Jun 19, 2024 3:37:50 PM com.azure.cosmos.implementation.Configs getAzureCosmosNonStreamingOrderByDisabled INFO: AZURE_COSMOS_DISABLE_NON_STREAMING_ORDER_BY env variable is: null

Do we need to show "AZURE_COSMOS_DISABLE_NON_STREAMING_ORDER_BY" in the logs?

Why is this not a Bug or a feature Request? I'm not sure if "AZURE_COSMOS_DISABLE_NON_STREAMING_ORDER_BY" need to be shown in the logs. If not, we need to update the code to remove it. https://github.com/Azure/azure-sdk-for-java/blob/34800a496de78f0f384698f031a17f7e0b2e35de/sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/implementation/Configs.java#L511

Setup (please complete the following information if applicable):

github-actions[bot] commented 2 weeks ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @kushagraThapar @pjohari-ms @TheovanKraay.

alexfandos commented 2 days ago

We are experiencing the same issue. It floods our logs. Does this also happen in 4.61.0? We will downgrade till this is fixed.

FabianMeiswinkel commented 2 days ago

This has been fixed in 4.62.0 - released on 2024-07-02.