[Internal] Use plugin framework implementation of share resource as default (#5089)

## Changes
<!-- Summary of your changes that are easy to understand -->
Make the plugin framework implementation of share resource as default.
With this change, `databricks_share` will use the plugin framework
implementation.

To be merged after:
#4969

Note: Removal of SDKv2 implementation will be done in next release so
users can fallback to SDKv2 implementation in case there are some
issues.

## Tests
<!--
How is this tested? Please see the checklist below and also describe any
other relevant tests
-->
Integration tests
- Applying: SDKv2 -> Plugin Framework 
- Applying: Plugin Framework -> SDKv2

Author: tanmay-db

NEXT_CHANGELOG.md

@@ -13,3 +13,5 @@
 ### Exporter
 
 ### Internal Changes
+
+* Make plugin framework implementation of share resource as default ([#4967](https://github.com/databricks/terraform-provider-databricks/pull/4967))

exporter/importables.go

@@ -2222,8 +2222,42 @@ var resourcesMap map[string]importable = map[string]importable{
 			return nil
 		},
 		Import: func(ic *importContext, r *resource) error {
+			resourceInfo := ic.Resources["databricks_share"]
+			if resourceInfo == nil {
+				// Fallback to direct data access if schema is not available
+				objectsList := r.Data.Get("object").([]any)
+				ic.emitUCGrantsWithOwner("share/"+r.ID, r)
+				for _, objRaw := range objectsList {
+					obj := objRaw.(map[string]any)
+					dataObjectType := obj["data_object_type"].(string)
+					name := obj["name"].(string)
+
+					switch dataObjectType {
+					case "TABLE":
+						ic.Emit(&resource{
+							Resource: "databricks_sql_table",
+							ID:       name,
+						})
+					case "VOLUME":
+						ic.Emit(&resource{
+							Resource: "databricks_volume",
+							ID:       name,
+						})
+					case "MODEL":
+						ic.Emit(&resource{
+							Resource: "databricks_registered_model",
+							ID:       name,
+						})
+					default:
+						log.Printf("[INFO] Object type '%s' (name: '%s') isn't supported in share '%s'",
+							dataObjectType, name, r.ID)
+					}
+				}
+				return nil
+			}
+
 			var share tf_sharing.ShareInfo
-			s := ic.Resources["databricks_share"].Schema
+			s := resourceInfo.Schema
 			common.DataToStructPointer(r.Data, s, &share)
 			// TODO: how to link recipients to share?
 			ic.emitUCGrantsWithOwner("share/"+r.ID, r)
@@ -2249,7 +2283,6 @@ var resourcesMap map[string]importable = map[string]importable{
 						obj.DataObjectType, obj.Name, r.ID)
 				}
 			}
-
 			return nil
 		},
 		ShouldOmitField: shouldOmitForUnityCatalog,

internal/providers/pluginfw/pluginfw_rollout_utils.go

@@ -33,11 +33,14 @@ import (
 var migratedResources = []func() resource.Resource{
 	library.ResourceLibrary,
 	qualitymonitor.ResourceQualityMonitor,
+	sharing.ResourceShare,
 }
 
 // List of data sources that have been migrated from SDK V2 to plugin framework
 // Keep this list sorted.
 var migratedDataSources = []func() datasource.DataSource{
+	sharing.DataSourceShare,
+	sharing.DataSourceShares,
 	volume.DataSourceVolumes,
 }
 
@@ -46,7 +49,6 @@ var migratedDataSources = []func() datasource.DataSource{
 var pluginFwOnlyResources = append(
 	[]func() resource.Resource{
 		app.ResourceApp,
-		sharing.ResourceShare,
 	},
 	autoGeneratedResources...,
 )
@@ -65,8 +67,6 @@ var pluginFwOnlyDataSources = append(
 		serving.DataSourceServingEndpoints,
 		// TODO: Add DataSourceCluster into migratedDataSources after fixing unit tests.
 		cluster.DataSourceCluster, // Using the staging name (with pluginframework suffix)
-		sharing.DataSourceShare,   // Using the staging name (with pluginframework suffix)
-		sharing.DataSourceShares,  // Using the staging name (with pluginframework suffix)
 	},
 	autoGeneratedDataSources...,
 )

internal/providers/pluginfw/products/sharing/data_share.go

@@ -28,7 +28,7 @@ type ShareDataSource struct {
 }
 
 func (d *ShareDataSource) Metadata(ctx context.Context, req datasource.MetadataRequest, resp *datasource.MetadataResponse) {
-	resp.TypeName = pluginfwcommon.GetDatabricksStagingName(dataSourceNameShare)
+	resp.TypeName = pluginfwcommon.GetDatabricksProductionName(dataSourceNameShare)
 }
 
 func (d *ShareDataSource) Schema(ctx context.Context, req datasource.SchemaRequest, resp *datasource.SchemaResponse) {

internal/providers/pluginfw/products/sharing/data_shares.go

@@ -52,7 +52,7 @@ type SharesDataSource struct {
 }
 
 func (d *SharesDataSource) Metadata(ctx context.Context, req datasource.MetadataRequest, resp *datasource.MetadataResponse) {
-	resp.TypeName = pluginfwcommon.GetDatabricksStagingName(dataSourceNameShares)
+	resp.TypeName = pluginfwcommon.GetDatabricksProductionName(dataSourceNameShares)
 }
 
 func (d *SharesDataSource) Schema(ctx context.Context, req datasource.SchemaRequest, resp *datasource.SchemaResponse) {

internal/providers/pluginfw/products/sharing/data_shares_acc_test.go

@@ -12,8 +12,8 @@ import (
 
 func checkSharesDataSourcePopulated(t *testing.T) func(s *terraform.State) error {
 	return func(s *terraform.State) error {
-		ds, ok := s.Modules[0].Resources["data.databricks_shares_pluginframework.this"]
-		require.True(t, ok, "data.databricks_shares_pluginframework.this has to be there")
+		ds, ok := s.Modules[0].Resources["data.databricks_shares.this"]
+		require.True(t, ok, "data.databricks_shares.this has to be there")
 		num_shares, _ := strconv.Atoi(ds.Primary.Attributes["shares.#"])
 		assert.GreaterOrEqual(t, num_shares, 1)
 		return nil
@@ -65,7 +65,7 @@ func TestUcAccDataSourceShares(t *testing.T) {
 			}
 		}
 
-		resource "databricks_share_pluginframework" "myshare" {
+		resource "databricks_share" "myshare" {
 			name = "{var.RANDOM}-terraform-delta-share"
 			object {
 				name = databricks_sql_table.mytable.id
@@ -81,8 +81,8 @@ func TestUcAccDataSourceShares(t *testing.T) {
 			}
 		}
 
-		data "databricks_shares_pluginframework" "this" {
-			depends_on = [databricks_share_pluginframework.myshare]
+		data "databricks_shares" "this" {
+			depends_on = [databricks_share.myshare]
 		}
 		`,
 		Check: checkSharesDataSourcePopulated(t),

internal/providers/pluginfw/products/sharing/resource_share.go

@@ -142,7 +142,7 @@ type ShareResource struct {
 }
 
 func (r *ShareResource) Metadata(ctx context.Context, req resource.MetadataRequest, resp *resource.MetadataResponse) {
-	resp.TypeName = pluginfwcommon.GetDatabricksStagingName(resourceName)
+	resp.TypeName = pluginfwcommon.GetDatabricksProductionName(resourceName)
 }
 
 func (r *ShareResource) Schema(ctx context.Context, req resource.SchemaRequest, resp *resource.SchemaResponse) {