weiqingy commented on issue #999: URL: https://github.com/apache/flink-agents/issues/999#issuecomment-5288871170
@kaiwangleo thanks for the work on this, and for taking on the CI part. That was the piece I most wanted to see fixed. #1001 already fixes this issue. The part that is still open is the CI gap. `ElasticsearchVectorStoreTest` is `@Disabled`, so none of this runs in CI today. That is genuinely useful work and it is yours. Would you be up for scoping #1011 down to just the CI enablement, on top of #1001? The two changes are independent, and splitting them lets a reviewer weigh the CI cost on its own merits. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
