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.

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

GSoC 2023: Aniket Agarwal week 8 #326

Merged
merged 3 commits into from
Jul 23, 2023

Conversation

Aniket-debug
Copy link
Contributor

@Aniket-debug Aniket-debug commented Jul 21, 2023

Changes proposed in this pull request:

  • updated following pagtap tests files:
  • no_crash_test
  • inner_query

@pgRouting/admins

@Aniket-debug Aniket-debug merged commit 1c5d8fb into pgRouting:Aniket-2023 Jul 23, 2023
@@ -45,8 +51,8 @@ subs TEXT[];
BEGIN
-- one to one
params = ARRAY[
'$$SELECT id, source, target, cost, reverse_cost FROM edge_table$$'
,'1::BIGINT',
'$$edges$$',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the original:

$$SELECT id, source, target, cost, reverse_cost FROM edge_table$$'

because edges is a prepared statement of the query above.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay got it.

Aniket-debug added a commit to Aniket-debug/GSoC-pgRouting that referenced this pull request Aug 10, 2023
* cleaning sql

* updated inner_qery.pg

* updated no_crash_test
Aniket-debug added a commit to Aniket-debug/GSoC-pgRouting that referenced this pull request Aug 22, 2023
* cleaning sql

* updated inner_qery.pg

* updated no_crash_test
Aniket-debug added a commit to Aniket-debug/GSoC-pgRouting that referenced this pull request Aug 23, 2023
* cleaning sql

* updated inner_qery.pg

* updated no_crash_test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants