AWS Bedrock Streaming Model implementation

[tomas1885]

When looking at the Bedrock Implementation, I see that only if the model is anthoric we use the default lanchain4j BedrockConverseStreamingChatModel, and by looking at the original PR this seems intentional, can anyone explain the rational behind this (@holomekc)? Was this a workaround to a specific issue? I tested a couple of other models and they seems to work fine with the default Langchain4j BedrockStreamingChatModel. If not remove it completely we can at-least make it configurable.
I don't mind opening a PR, just want to get the reasoning first.
@geoand

[geoand]

I'm on PTO so can't really look into this for the next few weeks

[geoand]

@tomas1885 I agree we should always default to LangChain4j. Do you want to open a PR?

[tomas1885]

I'm just contemplating on the configuration property name. As I want to keep the current behavior as the default so not to break the usage of anyone else, unless off-course we don't mind people would have to migrate to keep the old behavior

[geoand]

I don't think we need to keep the old behavior to be honest as I don't see what purpose it serves

[tomas1885]

So why not remove the if and just use the default langchain4j implementation, so no further configuration is needed?

[geoand]

Let's do that for now and see what happens