TwinCollection Max Depth Issue

[bouff]

• OS and version used:
  macOS 10.15.2
• Java runtime used:
  JDK 1.8
• SDK version used:
  iot-service-client version 1.19.0

Description of the issue:

As per the IoT Hub documentation, the depth limitation for desired properties, reported properties, and tag is 10.

https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-devguide-device-twins#tags-and-properties-format

The value is currently hard-coded at 6.

https://github.com/Azure/azure-iot-sdk-java/blob/master/deps/src/main/java/com/microsoft/azure/sdk/iot/deps/twin/TwinCollection.java

This is essentially a reincarnation of the following issue:
#338

Would it be possible to either update the value (or remove that validation, as discussed in the previous issue)?

Code sample exhibiting the issue:

Console log of the issue:

Need Support?

• Have a feature request for SDKs? Please post it on User Voice to help us prioritize.
• Have a technical question? Ask on Stack Overflow with tag “azure-iot-hub”
• Need Support? Every customer with an active Azure subscription has access to support with guaranteed response time. Consider submitting a ticket and get assistance from Microsoft support team
• Found a bug? Please help us fix it by thoroughly documenting it and filing an issue on GitHub (C, Java, .NET, Node.js, Python).

[timtay-microsoft]

Since the max depth appears to have changed from 5 to 10 from the service's perspective, it's probably a good idea for the SDK to just not care about the twin depth. We should definitely remove the code that validates that twin fits a certain depth

[eanavalentin]

Hi, is there an expected timeline for a new release which will include this fix?
We're also encountering this problem in production with an application which just stops execution and hangs on client connection to azure when we have reported properties with a depth of 7.

[timtay-microsoft]

I'm trying to get a new version of the SDK released with this fix by the end of this week. I'll update this thread once it is released

[timtay-microsoft]

As of this release, all SDK twin depth validation is removed, so I'm closing this issue

[az-iot-builder-01]

@bouff, @morphvale, thank you for your contribution to our open-sourced project! Please help us improve by filling out this 2-minute customer satisfaction survey