FWD » Core Development » Database

This issue was first reported in note 192 of #2068.

The following converts badly:

def temp-table tt1 field f1 as char.

for each tt1,
    first person where rowid(person) = to-rowid(tt1.f1):
end.

I think the FOR EACH, FIRST loop should convert to:

query0 = new CompoundQuery(false, false);
query0.addComponent(new AdaptiveQuery(tt1, (String) null, null, "tt1.id asc"));
RecordBuffer.prepare(person);
query0.addComponent(new RandomAccessQuery(person, "person.id = toRowid(?)", null, "person.siteId asc, person.empNum asc", new Object[]
{
   new FieldReference(tt1, "f1")
}), QueryConstants.FIRST);

Consider that:

def temp-table tt1 field f1 as rowid.

for each tt1,
    first person where rowid(person) = tt1.f1:
end.

already converts properly to:

query0 = new CompoundQuery(false, false);
query0.addComponent(new AdaptiveQuery(tt1, (String) null, null, "tt1.id asc"));
RecordBuffer.prepare(person);
query0.addComponent(new RandomAccessQuery(person, "person.id = ?", null, "person.siteId asc, person.empNum asc", new Object[]
{
   new FieldReference(tt1, "f1")
}), QueryConstants.FIRST);

The difference between the two is the data type of tt1.f1 and the use of the to-rowid builtin function in the where clause. I believe the latter is driving us down the path of a client-side where clause when we don't need to go there.