Skip to content

MB-65860: Added support for file callbacks#72

Closed
Likith101 wants to merge 3 commits intomasterfrom
fileCallbacks
Closed

MB-65860: Added support for file callbacks#72
Likith101 wants to merge 3 commits intomasterfrom
fileCallbacks

Conversation

@Likith101
Copy link
Copy Markdown
Member

No description provided.

Comment thread index.go Outdated
Copy link
Copy Markdown
Member

@abhinavdangeti abhinavdangeti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can drop this change and declare this in bleve - as it seems the only place where you use this.

@abhinavdangeti
Copy link
Copy Markdown
Member

abhinavdangeti commented Mar 25, 2026

@Likith101 do we need this change? It seems bleve is the only user for this interface - declare locally .. which already seem to have here - https://github.com/blevesearch/bleve/pull/2209/changes#diff-88ca3a9cf22d6ccb697576f30666f3c855089f0403a5709f05ae6067fe56c66eR392

Comment thread index.go
@Likith101 Likith101 closed this Mar 31, 2026
@github-project-automation github-project-automation Bot moved this from Todo to Done in File Callbacks Mar 31, 2026
@Likith101 Likith101 deleted the fileCallbacks branch March 31, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants