Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't unmarshal a partitioned cache #455

Open
chlowell opened this issue Sep 1, 2023 · 2 comments
Open

Can't unmarshal a partitioned cache #455

chlowell opened this issue Sep 1, 2023 · 2 comments
Labels
AzureSdk bug Something isn't working p2

Comments

@chlowell
Copy link
Collaborator

chlowell commented Sep 1, 2023

Easy to repro with a unit test in partitioned_storage_test.go:

func TestMarshal(t *testing.T) {
	mgr := newPartitionedManagerForTest(nil)
	b, err := mgr.Marshal()
	if err != nil {
		t.Fatal(err)
	}
	err = mgr.Unmarshal(b)
	if err != nil {
		t.Fatal(err)
	}
}

Unmarshal returns the error Unmarshal(*storage.InMemoryContract) only supports structs that have the field AdditionalFields or implements json.Unmarshaler. And that's spot on, storage.InMemoryContract doesn't have an AdditionalFields field.

@bgavrilMS bgavrilMS added bug Something isn't working p2 labels Nov 17, 2023
@bgavrilMS
Copy link
Member

Solved by #456

@chlowell
Copy link
Collaborator Author

Reopening because #456 didn't merge and it's still impossible to unmarshal a partitioned cache

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AzureSdk bug Something isn't working p2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants