Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature/corrections to oracle listagg implementation #408

Closed
wants to merge 43 commits into from
Closed

Feature/corrections to oracle listagg implementation #408

wants to merge 43 commits into from

Conversation

Chris53897
Copy link
Contributor

see #379

I do not have knowledge if the change is correct.
@olexp Can you please give some documentation here?

Chris8934 and others added 30 commits February 23, 2022 22:39
@Chris8934
chore: upgrade php-cs-fixer from v2 to v3
4b2fb2a
@Chris8934
fix: import for exception
5a2a2f9
@Chris8934
feat: small performance improvement
9f2f72a
@Chris8934
chore: fix ci deprecations
b1a661e
@Chris53897
Merge pull request #1 from Chris53897/feature/fix-ci-deprecation
d723238 
chore: fix ci deprecations
@Chris53897
Merge branch 'master' into feature/upgrade-php-cs-fixer-v3
be3efce
@Chris53897
Merge pull request #2 from Chris53897/feature/upgrade-php-cs-fixer-v3
d3ca2a3 
chore: upgrade php-cs-fixer from v2 to v3
@Chris53897
Merge pull request #3 from Chris53897/feature/improvments
d659c1b 
feat: small performance improvement
@Chris53897
Merge pull request #4 from Chris53897/feature/fix-exceptions
460736d 
fix: import for exception
@Chris8934
feat: modernize
e8b5d7f 
add symfony/cache, remove deprecations, drop support for php < 8.1
@Chris53897
Merge pull request #5 from Dukecity/feature/modernize
e6d32ff 
feat: modernize
@Chris8934
chore: remove cache in ci
1f23c9c
@Chris53897
Merge pull request #6 from Dukecity/feature/x
65ac0f8 
chore: remove cache in ci
@Chris8934
chore: apply cs-fixer-rules
3c4457b
@Chris8934
chore: run linter on ci for php 8.1
877fee5
@Chris53897
Merge pull request #7 from Dukecity/feature/lint
234f23d 
chore: apply-cs-fixer-rules
@Chris8934
feat: fix deprecation for md5
7728ce5
@Chris53897
Merge pull request #8 from Dukecity/feature/fix-deprecation-for-md5
b46cab9 
feat: fix deprecation for md5
@Chris8934
feat: fix deprecations, add some strong php types
7276645
@Chris53897
Merge pull request #9 from Dukecity/feature/fix-deprecations-add-stro…
6dcc205 
…ng-types

feat: fix deprecations, add some strong php types
@Chris8934
feat: Added REGEXP_REPLACE to Mysql
6077937
@Chris53897
Merge pull request #10 from Dukecity/feature/406-Added_REGEXP_REPLACE…
875f6ea 
…_to_Mysql

feat: Added REGEXP_REPLACE to Mysql
@Chris8934
feat: Add ILIKE to POSTGRES
449a94f
@Chris53897
Merge pull request #11 from Dukecity/feature/404-add_postgres_ilike
85796d1 
feat: Add ILIKE to POSTGRES
@Chris8934
feat: Add LAST_INSERT_ID to MYSQL
7834c31
@Chris53897
Merge pull request #12 from Dukecity/feature/Add_LAST_INSERT_ID-to-mysql
c77a7c1 
feat: Add LAST_INSERT_ID to MYSQL
@Chris8934
chore: update year of LICENSE
2981571
@Chris53897
Merge pull request #13 from Dukecity/feature/update-year-of-license
4d78f53 
chore: update year of LICENSE
@Chris8934
feat: add age() for POSTGRES
1f00a1e
@Chris53897
Merge pull request #14 from Dukecity/feature/add-age-for-postgres
e596c88 
feat: add age() for POSTGRES
Chris8934 and others added 13 commits June 16, 2023 08:35
@Chris8934
tests: Fix visibility for methods overriding TestCase::setUp() and Te…
c446e99 
…stCase::tearDown()
@Chris53897
Merge pull request #15 from Dukecity/feature/visibility-for-tests
b3e9c8f 
tests: Fix visibility for methods overriding TestCase::setUp()
@Chris8934
chore: fix formatting
e475b8d
@Chris53897
Merge pull request #16 from Dukecity/feature/fix-tests
507755d 
chore: fix formatting
@Chris8934
test: make test classes final
317367b
@Chris53897
Merge pull request #17 from Dukecity/feature/make-test-classes-final
634e1d9 
test: make test classes final
@Chris8934
test: make abstract the TestCase base classes
6aa7fff
@Chris53897
Merge pull request #18 from Dukecity/feature/abstract-testcase-base
25de52a 
test: make abstract the TestCase base classes
@Chris8934
feat: add quarter() for sqlite
4e666bc
@Chris53897
Merge pull request #19 from Dukecity/feature/add-quarterfor-sqlite
1a126b3 
feat: add quarter() for sqlite
@Chris8934
feat: add conv() for mysql
d3e9faa
@Chris53897
Merge pull request #20 from Dukecity/feature/add-conv-for-MySQL
5ca23e2 
feat: add conv() for mysql
@Chris8934
fix: listagg() for oracle
f4b32dc
@Chris53897 Chris53897 closed this Jun 16, 2023
@olexp
Copy link

olexp commented Jun 16, 2023 •
edited
Loading

Hello, Chris!
The log message for the 07815da commit already has it. But I'll put it here as well: https://docs.oracle.com/en/database/oracle/oracle-database/19/sqlrf/LISTAGG.html#GUID-B6E50D8E-F467-425B-9436-F7F8BF38D466

@Chris53897
Copy link
Contributor Author

Thanks. I added this link in my form for this function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@Chris53897 @olexp @Chris8934