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

Adding support for SchemaRegistryConfiguration(truststore password and location) and sasl jaas config in KafkaConfiguration #465

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GoncaloPT
Copy link

  • truststoreLocation
  • truststorePassword Adding support for KafkaConfiguration:
  • saslJaasConfig

Increasing MessageController maxOffset size since 1000 seems to be easily supported.

- truststoreLocation
- truststorePassword
Adding support for KafkaConfiguration:
- saslJaasConfig

Increasing MessageController maxOffset size since 1000 seems to be easily supported.
@GoncaloPT
Copy link
Author

GoncaloPT commented Jan 20, 2023

Adding support for SchemaRegistryConfiguration new parameters:

  • truststoreLocation
  • truststorePassword
    Adding support for KafkaConfiguration:
  • saslJaasConfig

Increasing MessageController maxOffset size since 1000 seems to be easily supported.
@ekoutanov

@GoncaloPT GoncaloPT changed the title Adding support for SchemaRegistryConfiguration new parameters: Adding support for SchemaRegistryConfiguration(truststore password and location) and sasl jaas config in KafkaConfiguration Jan 20, 2023
@davideicardi
Copy link
Collaborator

@GoncaloPT Thank you!

Can you please just use the previous code indentation configuration? It looks like many changes are only for different spaces ...
Also can you briefly describe these new parameters?

@GoncaloPT
Copy link
Author

@GoncaloPT Thank you!

Can you please just use the previous code indentation configuration? It looks like many changes are only for different spaces ... Also can you briefly describe these new parameters?

Hello @davideicardi. It was intellij defaults, but i will try to revert them and add some more description.

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.

2 participants