Skip to content

Conversation

idastambuk
Copy link
Contributor

This adds language (syntax highlighting and suggestions) support for the remainder of PPL commands:
joinCommand

  • renameCommand
  • statsCommand
  • grokCommand
  • patternsCommand
  • lookupCommand
  • kmeansCommand
  • adCommand
  • mlCommand
  • fillnullCommand
  • trendlineCommand
  • appendcolCommand
  • expandCommand
  • flattenCommand
  • reverseCommand

@idastambuk idastambuk requested a review from a team as a code owner September 2, 2025 15:23
@idastambuk idastambuk requested review from iwysiu and njvrzm and removed request for a team September 2, 2025 15:23
@idastambuk idastambuk linked an issue Sep 2, 2025 that may be closed by this pull request
15 tasks
Copy link
Contributor

@kevinwcyu kevinwcyu left a comment

Choose a reason for hiding this comment

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

looks good, just wondering about the join query suggestions in the comment below.


export const joinQuery = {
query:
'left outer join left = table1 right = table2 left_hint.id = leftIdent on isnull(leftIdent) = false right = table2',
Copy link
Contributor

Choose a reason for hiding this comment

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

is left supposed to be suggested? when i type left i don't get that as a suggestion, it also isn't followed by outer as a suggestion.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I left that out of the suggestions cause I thought it could be confusing bundling it with the commands. But it makes sense to suggest outer and join after left/right, will add that

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@idastambuk idastambuk enabled auto-merge (squash) September 8, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PPL Syntax Support ph.2: Add commands to ppl grammar
2 participants