Skip to content

add python schema field default value #8122

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Oct 9, 2020
Merged

add python schema field default value #8122

merged 5 commits into from
Oct 9, 2020
+71 −1

Conversation

hangc0276
Copy link
Contributor

Fix #7673

Changes

For each python defined field type, init default value.

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

2 similar comments
@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@BewareMyPower
Copy link
Contributor

I've fixed the testIndividualAck cpp unit test failure in #8141 , please rebase and commit again, thx!

@hangc0276 hangc0276 force-pushed the python_schema_field_default_value branch from 228734f to 10ba66e Compare September 28, 2020 00:45
@hangc0276
Copy link
Contributor Author

I've fixed the testIndividualAck cpp unit test failure in #8141 , please rebase and commit again, thx!

Thx, I will rebase the master.

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@sijie
Copy link
Member

sijie commented Oct 1, 2020

/pulsarbot run-failure-checks

@sijie sijie added this to the 2.7.0 milestone Oct 1, 2020
@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

1 similar comment
@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

6 similar comments
@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@skyrocknroll
Copy link
Contributor

@shiv4289 ^^

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

2 similar comments
@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@codelipenghui codelipenghui merged commit e3336fa into apache:master Oct 9, 2020
wolfstudy pushed a commit that referenced this pull request Oct 30, 2020
@hangc0276 @wolfstudy
Ddd python schema field default value (#8122)
be44bf8 
Fix #7673 

### Changes
For each python defined field type, init default value.

(cherry picked from commit e3336fa)
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Nov 13, 2020
@hangc0276 @huangdx0726
Ddd python schema field default value (apache#8122)
8f9861a 
Fix apache#7673 

### Changes
For each python defined field type, init default value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sijie sijie

@codelipenghui codelipenghui

Assignees

@hangc0276 hangc0276

Labels
release/2.6.2
Projects
None yet
Milestone
2.7.0
Development

Successfully merging this pull request may close these issues.

Consumer error on schema boolean field validation - Python client
5 participants
@hangc0276 @BewareMyPower @sijie @skyrocknroll @codelipenghui