Eric Hare
Eric Hare
Thanks @potter-potter !
@potter-potter just tried to address your comments. agree with all of them and explained a little for the prior (misguided, lol) motivation :) let me know if this looks better!
> @erichare This is looking good. I can take over once you make the little dict change. Thanks @potter-potter ! I made the update, does it look okay?
> @erichare Looking good! I'll bring it to the finish line tomorrow. Thanks! Thank you very much!
> @erichare I'm taking a look at this. > > Can you give me a description on why someone would use Astra DB as a source connector? We don't have...
@nicoloboschi i made the changes you listed, let me know if this looks better. Thank you!
@potter-potter i had something come up earlier and haven't had a chance to fully test, i will get back to you on that soon. Also sorry for the very undescriptive...
Gotchya. I tried to pull out the logic that was specific (i.e., metadata fields, we dont really have a great notion of), but the basic idea of pulling the data,...
> It actually looks like this gets the entire table: astra_docs = list(self.astra_db_collection.paginated_find()) > > In most of the connectors, since they are not table based, `get_ingest_docs` just gets a...
@potter-potter I think I tracked down the issue. This was my mistake - i passed just the single field `metadata`, but actually the API expects a list of fields even...