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

Imports functions #11

Merged
merged 2 commits into from
May 9, 2023
Merged

Imports functions #11

merged 2 commits into from
May 9, 2023

Conversation

huangdijia
Copy link
Contributor

No description provided.

@huangdijia huangdijia merged commit d701dc2 into 4.x May 9, 2023
@huangdijia huangdijia deleted the insteads branch May 9, 2023 00:16
huangdijia added a commit that referenced this pull request Jun 12, 2023
* Increase concurrent limit and adjust Replication timeout

- Increase the concurrent limit for publishing triggers
- Change the `consume()` call to `wait()` with a timeout parameter

* refactor: Rename

* Update trigger and subscriber connections

- Update `trigger` and `subscriber` annotations to use `connection` instead of `pool`
- Update `TriggerManager` and `SubscriberManager` to use `connection` instead of `pool`
- Update `HealthMonitor`, `TriggerSubscriber`, `RedisBinLogCurrentSnapshot`, `RedisServerMutex`, `ConsumerManager` and `Consumer` to use `connection` instead of `pool`
- Update `Trig

* Improve SubscribersCommand output and fix typo

- Fix typos in the info message of the SubscribersCommand
- Update table headers to use proper case for `Connection`

* Improve Subscribers Command filtering logic

- Update the filter logic of the `SubscribersCommand` to check for the `connection` option
- Use the `$connection` variable instead of `$this->input->getOption('connection')`

* Improve connection and snapshot handling

- Fix typos in method names from `getconnection()` to `getConnection()`
- Set `health_monitor.interval` and `snapshot.interval` to `10`
- Set `binLogCurrentSnapshot` to the value of the consumer

* Enable consumer processes in Publish Trigger

- Add `enable` parameter to `default` connection
- Skip connections if `enable` is set to `false`
- Add a process for each consumer with the process name set to the consumer name and the process number set to `1`
- Register the process with the ProcessManager

* Format

* Format

---------

Optimize

Bumps hyperf/* to v3.1

Imports functions (#11)

* Imports functions

* 🛠️refactor(scope) restructure file imports in ConfigProvider.php

---------

Adds hyperf/context and hyperf/coroutine packages to composer.json. (#10)

* Adds hyperf/context and hyperf/coroutine packages to composer.json.

* 🚀feat(Consumer.php) update Coroutine import

* ⚙️chore(consumer) import wait function from Coroutine

* Bumps packages

* Remove `hyperf/utils`

---------

Adds ConsumeCommand (#9)

* Adds ConsumeCommand

* ⚙️chore(dependencies) add Hyperf Tappable and Collection packages

* ⚙️chore(dependencies) add Hyperf Collection\Arr to files

---------

Improve health monitoring and utility functions (#8)

- Add a check to ensure EventDispatcherInterface is present before dispatching the OnReplicationStop event
- Change usage of binLogCurrentSnapshot to `$binLogCurrentCache` for better readability
- Ensure binLogCurrentSnapshot and binLogCurrent have the same binLogPosition before dispatching the OnReplicationStop event
- Add `use RuntimeException` statement
- Change `\RuntimeException` to `RuntimeException` in the `getInternalIp

Improve Consumer Manager and Consumer classes (#7)

- Change the constructor of the AbstractProcess class to include a ContainerInterface parameter
- Update debug and warning messages in the Consumer class

Adds BinaryDataReaderAspect (#6)

* Improve configuration and synchronization

- Add Redis implementations for ServerMutexInterface and BinLogCurrentSnapshotInterface
- Add BinaryDataReaderAspect to aspects configuration

* Configure dev-master branch alias

- Add a `branch-alias` configuration for the `dev-master` branch in `composer.json`

---------

Co-Authored-By: Deeka Wong <[email protected]>
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.

None yet

1 participant