-
Notifications
You must be signed in to change notification settings - Fork 49
rebuild fulltext indexes via listener #482
rebuild fulltext indexes via listener #482
Conversation
552d43c
to
65c9782
Compare
c4c1853
to
c3b4429
Compare
fcfd64d
to
5a3a3c3
Compare
be933e6
to
db8ccd5
Compare
depend on : |
The branch push some weird commit, you may misuse the branch. |
Good catch! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally LGTM, consider add a test case in NebulaListenerTest to test the snapshot, maybe need to set the size of wal_file and wal_ttl to be very small.
Yes, the snapshot test case should have been added a long time ago (since PR #465 submitted), for clarity, let me add it in the new PR. please wait a moment. |
ok |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job!
bae258d
8cfc17c
to
bae258d
Compare
fixed compiler warning for
|
No description provided.