Skip to content

FIX: [environment] change the semantic of sync symbols with prefix#2410

Merged
dboyliao merged 1 commit intomainfrom
dboy/fix/profit-fixer
Mar 4, 2026
Merged

FIX: [environment] change the semantic of sync symbols with prefix#2410
dboyliao merged 1 commit intomainfrom
dboy/fix/profit-fixer

Conversation

@dboyliao
Copy link
Copy Markdown
Collaborator

@dboyliao dboyliao commented Mar 4, 2026

This PR changes the behavior of sync symbols with prefixes to be session-specific, rather than additive to the global sync symbols.

Changes

Before:

  • All sessions would sync the "rest symbols" (symbols without prefix) by default
  • Session-specific symbols (with prefix like max:BTCUSDT) would be appended to the rest symbols
  • This meant sessions with specific symbols would sync both their specific symbols AND all the rest symbols

After:

  • Sessions with specific symbols defined will ONLY sync those symbols
  • Sessions without specific symbols will sync all the "rest symbols"
  • This provides better control and isolation for session-specific sync behavior

Example

If you have:

sync:
  symbols:
    - BTCUSDT
    - ETHUSDT
    - binance:BNBUSDT

Before: The binance session would sync BTCUSDT, ETHUSDT, and BNBUSDT
After: The binance session will sync only BNBUSDT, while other sessions sync BTCUSDT and ETHUSDT

@dboyliao dboyliao requested a review from c9s as a code owner March 4, 2026 04:40
…o session specific sync symbols

- the session with such symbols defined will only sync those symbols
- sync symbols without prefix will be applied to all other sessions which have no session specific sync symbols defined
@dboyliao dboyliao force-pushed the dboy/fix/profit-fixer branch from 25a85d7 to b8c1dd1 Compare March 4, 2026 05:27
@dboyliao dboyliao merged commit dc89dc3 into main Mar 4, 2026
3 checks passed
@dboyliao dboyliao deleted the dboy/fix/profit-fixer branch March 4, 2026 06:16
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