Fernando Raganhan Barbosa

Implement ORDER BY

10

Just like the title says: add support for ORDER BY according to the specification.

Introduce .internal packages

1

The public API of the interpreter exposes a lot of components publicly without explicit intention. Part of the reason for this is that we sometimes rely on Kotlin's `internal` access...

Should fail for symbols outside of the LST

2

See * https://github.com/amzn/ion-tests/blob/master/iontestdata/bad/annotationSymbolIDUnmapped.ion * https://github.com/amzn/ion-tests/blob/master/iontestdata/bad/annotationSymbolIDUnmapped.10n * https://github.com/amzn/ion-tests/blob/master/iontestdata/bad/fieldNameSymbolIDUnmapped.ion * https://github.com/amzn/ion-tests/blob/master/iontestdata/bad/fieldNameSymbolIDUnmapped.10n

Ion-test: ion-tests/iontestdata/good/typecodes/T6-large.10n See this build https://travis-ci.org/amzn/ion-js/builds/644138291?utm_source=github_status&utm_medium=notification It passes for Node12 but not 10 and 8

ion-tests/iontestdata/good/equivs/longStringsWithComments.ion

ion-tests/iontestdata/good/equivs/localSymbolTableWithAnnotations.ion

ion-tests/iontestdata/good/equivs/clobNewlines.ion

See * ion-tests/iontestdata/good/typecodes/T7-large.10n * ion-tests/iontestdata/good/typecodes/T7-small.10n

Lists, s-exp and structs * ion-tests/iontestdata/good/typecodes/T11.10n * ion-tests/iontestdata/good/typecodes/T12.10n * ion-tests/iontestdata/good/typecodes/T13.10n

http://amzn.github.io/ion-docs/docs/binary.html#6-timestamp