[AUTO] Increment version to 3.2.0-SNAPSHOT

[opensearch-trigger-bot]

• Incremented version to 3.2.0-SNAPSHOT.

[dbwiddis]

Depends on:

• [AUTO] Increment version to 3.2.0-SNAPSHOT common-utils#844
• [AUTO] Increment version to 3.2.0-SNAPSHOT opensearch-remote-metadata-sdk#209
• [AUTO] Increment version to 3.2.0-SNAPSHOT ml-commons#3942
• [AUTO] Increment version to 3.2.0-SNAPSHOT k-NN#2776
• [AUTO] Increment version to 3.2.0-SNAPSHOT neural-search#1406
• Add Apache commons-text as we can't access ML Commons plugin classpath #1195

[dbwiddis]

Failing due to:

java.lang.ClassNotFoundException: org.apache.commons.text.StringSubstitutor

This is another issue with ML Commons including a dependency that we need (MLCreateConnectorInput invokes it) but has a different classpath due to being in a different plugin. Seems opensearch-project/ml-commons#3579 introduced this.

See also opensearch-project/ml-commons#3860 for related issue.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.66%. Comparing base (4794ac2) to head (3762e3a).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##               main    #1178   +/-   ##
=========================================
  Coverage     77.66%   77.66%           
  Complexity     1192     1192           
=========================================
  Files           102      102           
  Lines          5641     5641           
  Branches        574      574           
=========================================
  Hits           4381     4381           
  Misses          986      986           
  Partials        274      274           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[joshpalis]

Holding off on merging until #1195 is merged

[dbwiddis]

New error.

/home/runner/work/flow-framework/flow-framework/src/main/java/org/opensearch/flowframework/workflow/AbstractRegisterLocalModelStep.java:31: error: import requires canonical name for FrameworkType
import org.opensearch.ml.common.model.TextEmbeddingModelConfig.FrameworkType;

This is a public enum in a public class. Or was last time I downloaded the class.

Looks like in the latest version the enum has moved to the superclass org.opensearch.ml.common.model.BaseModelConfig so is accessible from the subclass but not from imports from dependencies.

https://docs.oracle.com/javase/specs/jls/se8/html/jls-7.html#jls-7.5.1

Gotta love breaking changes in minor version bumps! :)

Change introduced in opensearch-project/ml-commons#3994 (CC: @nathaliellenaa @b4sjoo @Zhangxunmt please confirm my fix is the correct one)

[nathaliellenaa]

New error.

/home/runner/work/flow-framework/flow-framework/src/main/java/org/opensearch/flowframework/workflow/AbstractRegisterLocalModelStep.java:31: error: import requires canonical name for FrameworkType
import org.opensearch.ml.common.model.TextEmbeddingModelConfig.FrameworkType;

This is a public enum in a public class. Or was last time I downloaded the class.

Looks like in the latest version the enum has moved to the superclass org.opensearch.ml.common.model.BaseModelConfig so is accessible from the subclass but not from imports from dependencies.

https://docs.oracle.com/javase/specs/jls/se8/html/jls-7.html#jls-7.5.1

Gotta love breaking changes in minor version bumps! :)

Change introduced in opensearch-project/ml-commons#3994 (CC: @nathaliellenaa @b4sjoo @Zhangxunmt)

@dbwiddis Yep, it should be changed to import org.opensearch.ml.common.model.BaseModelConfig.FrameworkType;. Your changes here looks correct!