openlookeng / hetu-core

570 stars 423 forks source link

added Kryo Integration for Spiller Serialization #278

Closed Nitin-Kashyap closed 2 years ago

Nitin-Kashyap commented 2 years ago

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line: /kind feature

What does this PR do / why do we need it:

Integrated Kryo stream serialization:

Additionally, Replaced Mockito-All with Mokito-Core as it conflicts with Kryo for Objenesis jars. Also, standardized the version used for Mockito-core to 1.10.19 which matches the one used in mockito-all 1.10.18. Note: Some of the versions of mockito-all used in some of the projects conflicts with other wherein, some interfaces were removed in later versions. Hence, standardization to 1.10.19 was found to be safe in all the usages of all projects as of the time of this PR

Which issue(s) this PR fixes:

Fixes #277

Special notes for your reviewers:

it-is-a-robot commented 2 years ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Nitin-Kashyap To complete the pull request process, please assign lizheng920625 You can assign the PR to them by writing /assign @lizheng920625 in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/openlookeng/hetu-core/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
sraghunandan commented 2 years ago

/sync

it-is-a-robot commented 2 years ago

@sraghunandan: This pr has been synchronized to the Gitee Repository

In response to [this](https://github.com/openlookeng/hetu-core/pull/278#issuecomment-1025377520): >/sync Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [opensourceways/test-infra](https://github.com/opensourceways/test-infra/issues/new?title=Prow%20issue:) repository.
sraghunandan commented 2 years ago

/sync

it-is-a-robot commented 2 years ago

@sraghunandan: This pr has been synchronized to the Gitee Repository

In response to [this](https://github.com/openlookeng/hetu-core/pull/278#issuecomment-1025602304): >/sync Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [opensourceways/test-infra](https://github.com/opensourceways/test-infra/issues/new?title=Prow%20issue:) repository.
sraghunandan commented 2 years ago

/sync

it-is-a-robot commented 2 years ago

@sraghunandan: This pr has been synchronized to the Gitee Repository

In response to [this](https://github.com/openlookeng/hetu-core/pull/278#issuecomment-1026476131): >/sync Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [opensourceways/test-infra](https://github.com/opensourceways/test-infra/issues/new?title=Prow%20issue:) repository.
sraghunandan commented 2 years ago

/sync

it-is-a-robot commented 2 years ago

@sraghunandan: This pr has been synchronized to the Gitee Repository

In response to [this](https://github.com/openlookeng/hetu-core/pull/278#issuecomment-1026517647): >/sync Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [opensourceways/test-infra](https://github.com/opensourceways/test-infra/issues/new?title=Prow%20issue:) repository.
sraghunandan commented 2 years ago

/sync

Nitin-Kashyap commented 2 years ago

/sync

sraghunandan commented 2 years ago

/sync

it-is-a-robot commented 2 years ago

@sraghunandan: This pr has been synchronized to the Gitee Repository

In response to [this](https://github.com/openlookeng/hetu-core/pull/278#issuecomment-1032205487): >/sync Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [opensourceways/test-infra](https://github.com/opensourceways/test-infra/issues/new?title=Prow%20issue:) repository.
sraghunandan commented 2 years ago

/sync

it-is-a-robot commented 2 years ago

@sraghunandan: This pr has been synchronized to the Gitee Repository

In response to [this](https://github.com/openlookeng/hetu-core/pull/278#issuecomment-1043875224): >/sync Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [opensourceways/test-infra](https://github.com/opensourceways/test-infra/issues/new?title=Prow%20issue:) repository.
sraghunandan commented 2 years ago

/sync

it-is-a-robot commented 2 years ago

@sraghunandan: This pr has been synchronized to the Gitee Repository

In response to [this](https://github.com/openlookeng/hetu-core/pull/278#issuecomment-1044370108): >/sync Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [opensourceways/test-infra](https://github.com/opensourceways/test-infra/issues/new?title=Prow%20issue:) repository.
sraghunandan commented 2 years ago

/sync

it-is-a-robot commented 2 years ago

@sraghunandan: This pr has been synchronized to the Gitee Repository

In response to [this](https://github.com/openlookeng/hetu-core/pull/278#issuecomment-1046465366): >/sync Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [opensourceways/test-infra](https://github.com/opensourceways/test-infra/issues/new?title=Prow%20issue:) repository.
sraghunandan commented 2 years ago

/lgtm

sraghunandan commented 2 years ago

/approve