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

SandTable on Kafka #1

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

Conversation

Qian-Cheng-nju
Copy link

Kafka was deployed in SandTable, message deserialization and hooks for seven I/O-related calls include fread and fwrite were added.

@tangruize
Copy link
Owner

Thank you for the contribution!

The PR needs some improvements.

  1. Please remove unnecessary files like .DS_STORE
  2. Avoid modifying files that are unrelated to the Kafka systems.
  3. After making these changes, please verify that Kafka runs properly and that the other systems remain functional.

If this requires significant effort, we can consider adding Kafka as a branch temporarily.

@Qian-Cheng-nju
Copy link
Author

Hi tangruize,

Thank you for your detailed feedback and suggestions.

I will remove the unnecessary files like .DS_STORE and ensure that no unrelated files to the Kafka systems are modified. I will also verify that Kafka runs properly and that the other systems remain functional.

If there are any further issues or if you need additional changes, please let me know. I appreciate your guidance and cooperation.

Best regards,
Rogue-Link

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