meta: fix remove resource group

meta/meta.go

@@ -574,10 +574,7 @@ func (m *Meta) UpdateResourceGroup(group *model.ResourceGroupInfo) error {
 func (m *Meta) DropResourceGroup(groupID int64) error {
 	// Check if group exists.
 	groupKey := m.resourceGroupKey(groupID)
-	if err := m.txn.HClear(groupKey); err != nil {
-		return errors.Trace(err)
-	}
-	if err := m.txn.HDel(mPolicies, groupKey); err != nil {
+	if err := m.txn.HDel(mResourceGroups, groupKey); err != nil {
 		return errors.Trace(err)
 	}
 	return nil

meta/meta_test.go

@@ -101,6 +101,45 @@ func TestPlacementPolicy(t *testing.T) {
 	require.NoError(t, err)
 }
 
+func TestResourceGroup(t *testing.T) {
+	store, err := mockstore.NewMockStore()
+	require.NoError(t, err)
+
+	defer func() {
+		require.NoError(t, store.Close())
+	}()
+
+	txn, err := store.Begin()
+	require.NoError(t, err)
+
+	// test the independent policy ID allocation.
+	m := meta.NewMeta(txn)
+
+	checkResourceGroup := func(ru uint64) {
+		rg, err := m.GetResourceGroup(1)
+		require.NoError(t, err)
+		require.Equal(t, rg.RURate, ru)
+	}
+
+	rg := &model.ResourceGroupInfo{
+		ID:   1,
+		Name: model.NewCIStr("aa"),
+		ResourceGroupSettings: &model.ResourceGroupSettings{
+			RURate: 100,
+		},
+	}
+	require.NoError(t, m.CreateResourceGroup(rg))
+	checkResourceGroup(100)
+
+	rg.RURate = 200
+	require.NoError(t, m.UpdateResourceGroup(rg))
+	checkResourceGroup(200)
+
+	m.DropResourceGroup(1)
+	_, err = m.GetResourceGroup(1)
+	require.Error(t, err)
+}
+
 func TestBackupAndRestoreAutoIDs(t *testing.T) {
 	store, err := mockstore.NewMockStore()
 	require.NoError(t, err)