GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Join.java 中 SQLConfig outerConfig 重命名成 onConfig,然后用原来的 outerConfig 增加最外层的where 查询条件 by wz11wz · Pull Request #829 · APIJSON/APIJSON

Conversation

@wz11wz

@wz11wz wz11wz commented Jul 18, 2025

Copy link
Copy Markdown
Contributor

AbstractSQLConfig gainWhereString 增加适配逻辑.
1.inner join 本身是最外面增加条件，忽略这种情况
2.onConfig       "@group": "key2+"  // JOIN 外层 GROYP BY key2
                 "@having": "count(key3)>0"  // JOIN 外层 HAVING count(key3)>0
                 "@order": "key2+,key3-"  // JOIN 外层 ORDER BY key2 ASC, key3 DESC  本身这些条件就是加到最外层。

"join": {
  "</Location_info/code@": {
    "id": "88888",
    "@column": "key2,key3",
    "@group": "key2+",
    "@having": "count(key3)>0",
    "@order": "key2+,key3-"
  },
  "id": "88888",只有这个条件加 on 上.   其他        "@group": "key2+","@having": "count(key3)>0", "@order": "key2+,key3-" 都是加到 最外面的where条件上.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
TommyLemon reacted with thumbs up emoji TommyLemon reacted with heart emoji
All reactions
  • 👍 1 reaction
  • ❤️ 1 reaction
wz11wz and others added 3 commits July 18, 2025 16:39
@wz11wz
fix: 现在的 Join.java 中 SQLConfig outerConfig 重命名成 onConfig 及对应的 get set…
3c21439 
… 方法，然后再加回来 SQLConfig outerConfig，然后下方的键值对放到 outerConfig 上，后面再具体 AbstractSQLConfig gainWhereString。

1、inner join 本身是最外面增加条件，没有增加
2、onConfig       "@group": "key2+"  // JOIN 外层 GROYP BY key2
                 "@having": "count(key3)>0"  // JOIN 外层 HAVING count(key3)>0
                 "@order": "key2+,key3-"  // JOIN 外层 ORDER BY key2 ASC, key3 DESC  本身这些条件就是加到最外层。
@wz11wz
Merge remote-tracking branch 'origin/master'
b05016f
@TommyLemon
优化 AbstractParser.java 缩进
2122b8c
TommyLemon
TommyLemon approved these changes Jul 20, 2025
View reviewed changes
@wz11wz

wz11wz commented Jul 20, 2025 via email

Copy link
Copy Markdown
Contributor Author
这是来自QQ邮箱的假期自动回复邮件。您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@TommyLemon
TommyLemon merged commit 4161902 into APIJSON:master Jul 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TommyLemon TommyLemon TommyLemon approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@wz11wz @TommyLemon
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.