Make this function more efficient by using a single query to get all the rowids in the table using one query

[github-actions]

Currently this function is called every time to fetch the boundary, instead of this a change is expected here that can fetch all the rowids at once, this might significantly improve the query performance and reduce repeated db calls.

olake/drivers/oracle/internal/backfill.go

Lines 90 to 95 in 8804519

	// TODO: Make this function more efficient by using a single query to get all the rowids in the table using one query
	func (o *Oracle) splitViaTableIteration(ctx context.Context, stream types.StreamInterface, avgRowSize float64) (*types.Set[types.Chunk], error) {
	chunks := types.NewSet[types.Chunk]()
	var minRowId, maxRowId string
	query := jdbc.OracleMinMaxRowIDQuery(stream)

---

This issue was generated by todo-issue based on a TODO comment in 8804519. It's been assigned to @vaibhav-datazip because they committed the code.

[mvanhorn]

I've submitted a PR (#914) that replaces the N+1 queries in splitViaTableIteration with a single NTILE-based query. Would appreciate the assignment.

[github-actions]

Closing Issue

This comment was generated by todo-issue based on the deletion of a TODO comment in 30704c1.