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

Dart implementation fix - the object storage client aren't serializable #47

Open
wants to merge 3 commits into
base: cosDataStore
Choose a base branch
from

Conversation

rajuGT
Copy link

@rajuGT rajuGT commented Oct 23, 2024

Most of the client implementation including GCS, is not serializable, so fixed this issue by making client implementation not part of the serialization, and when the client is passed over wire and the client doesn't exist, it initializes as and when it is required.

    // In a distributed system, we don't intend the client to be serialized and most of the implementations like
    // GCP Storage implementation doesn't implement java.io.Serializable interface and you may see the below error
    // Caused by: org.apache.flink.api.common.InvalidProgramException: com.google.api.services.storage.Storage@1c666a8f
    // is not serializable. The object probably contains or references non serializable fields.
    // Caused by: java.io.NotSerializableException: com.google.api.services.storage.Storage

rajuGT added 3 commits October 23, 2024 22:43
Most of the client implementation including GCS, is not serializable, so
fixed this issue by making client implementation not part of the
serialization, and when the client is passed over wire and the client
doesn't exist, it initializes as and when it is required.

        // In a distributed system, we don't intend the client to be serialized and most of the implementations like
        // GCP Storage implementation doesn't implement java.io.Serializable interface and you may see the below error
        // Caused by: org.apache.flink.api.common.InvalidProgramException: com.google.api.services.storage.Storage@1c666a8f
        // is not serializable. The object probably contains or references non serializable fields.
        // Caused by: java.io.NotSerializableException: com.google.api.services.storage.Storage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant