Skip to content

Make KeyValueSchema an interface visible in the public Schema API - #10888

Merged
eolivelli merged 4 commits into
apache:masterfrom
eolivelli:impl/keyvalueschema-interface
Jun 10, 2021
Merged

Make KeyValueSchema an interface visible in the public Schema API#10888
eolivelli merged 4 commits into
apache:masterfrom
eolivelli:impl/keyvalueschema-interface

Conversation

@eolivelli

@eolivelli eolivelli commented Jun 10, 2021

Copy link
Copy Markdown
Contributor
  • allow users of pulsar-client-api to use KeyValueSchema
  • move KeyValueSchema implementation to KeyValueSchemaImpl
  • introduce a new interface org.apache.pulsar.client.api.schema.KeyValueSchema

Verifying this change

The integration test file TestGenericObjectSink has been changed to use the new API class.
See the comments in #10878

- allow users of pulsar-client-api to use KeyValueSchema
- move KeyValueSchema implementation to KeyValueSchemaImpl
- introduce a new interface KeyValueSchema
@eolivelli
eolivelli requested a review from merlimat June 10, 2021 07:20
@eolivelli eolivelli self-assigned this Jun 10, 2021
@eolivelli eolivelli added the release/blocker Indicate the PR or issue that should block the release until it gets resolved label Jun 10, 2021
@eolivelli eolivelli added this to the 2.8.0 milestone Jun 10, 2021
Comment thread pulsar-client-api/src/main/java/org/apache/pulsar/client/api/KeyValueSchema.java Outdated
@eolivelli
eolivelli requested a review from sijie June 10, 2021 07:45

@lhotari lhotari left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@eolivelli
eolivelli merged commit 18f2f4a into apache:master Jun 10, 2021
@eolivelli
eolivelli deleted the impl/keyvalueschema-interface branch June 10, 2021 09:56
eolivelli added a commit to datastax/pulsar that referenced this pull request Jun 10, 2021
…ache#10888)

* Make KeyValueSchema an interface visible in the public Schema API
- allow users of pulsar-client-api to use KeyValueSchema
- move KeyValueSchema implementation to KeyValueSchemaImpl
- introduce a new interface KeyValueSchema

(cherry picked from commit 18f2f4a)
eolivelli added a commit to datastax/pulsar that referenced this pull request Jun 10, 2021
…ache#10888)

* Make KeyValueSchema an interface visible in the public Schema API
- allow users of pulsar-client-api to use KeyValueSchema
- move KeyValueSchema implementation to KeyValueSchemaImpl
- introduce a new interface KeyValueSchema

(cherry picked from commit 18f2f4a)
codelipenghui pushed a commit that referenced this pull request Jun 11, 2021
…0888)

* Make KeyValueSchema an interface visible in the public Schema API
- allow users of pulsar-client-api to use KeyValueSchema
- move KeyValueSchema implementation to KeyValueSchemaImpl
- introduce a new interface KeyValueSchema

(cherry picked from commit 18f2f4a)
eolivelli added a commit to datastax/pulsar that referenced this pull request Jun 14, 2021
…ache#10888)

* Make KeyValueSchema an interface visible in the public Schema API
- allow users of pulsar-client-api to use KeyValueSchema
- move KeyValueSchema implementation to KeyValueSchemaImpl
- introduce a new interface KeyValueSchema

(cherry picked from commit 18f2f4a)
(cherry picked from commit 1ecadd4)
yangl pushed a commit to yangl/pulsar that referenced this pull request Jun 23, 2021
…ache#10888)

* Make KeyValueSchema an interface visible in the public Schema API
- allow users of pulsar-client-api to use KeyValueSchema
- move KeyValueSchema implementation to KeyValueSchemaImpl
- introduce a new interface KeyValueSchema
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
…ache#10888)

* Make KeyValueSchema an interface visible in the public Schema API
- allow users of pulsar-client-api to use KeyValueSchema
- move KeyValueSchema implementation to KeyValueSchemaImpl
- introduce a new interface KeyValueSchema
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release/blocker Indicate the PR or issue that should block the release until it gets resolved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants