gortiz opened a new pull request, #13793:
URL: https://github.com/apache/pinot/pull/13793
This is a simple PR that adds the `@Language("sql")` annotation on most test
method arguments.
When used, this annotation indicates Intellij that the caller string should
be a string and therefore it is highlighted as such. For example:
Notice: By default Intellij also shows a warning in these strings because it
expects to have a datasource configured. This is because this feature is
focused on web devs that always attack the same database schema. If a
datasource is provided, Intellij can detect more errors (like databases that
doesn't exist or type errors). In our case, where most our sql strings are used
in tests where the schema is generated at runtime, we cannot use that feature,
so it is recommended to disable the hint.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
