TheNeuralBit commented on a change in pull request #13565:
URL: https://github.com/apache/beam/pull/13565#discussion_r568708871
##########
File path: website/www/site/content/en/blog/adding-data-sources-to-sql.md
##########
@@ -81,19 +81,19 @@ The `TableProvider` classes are under
Our table provider looks like this:
-{{< highlight java >}}
+{{< highlight language="java" >}}
@AutoService(TableProvider.class)
public class GenerateSequenceTableProvider extends InMemoryMetaTableProvider {
- @Override
- public String getTableType() {
- return "sequence";
- }
+@Override
+public String getTableType() {
+return "sequence";
+}
- @Override
- public BeamSqlTable buildBeamSqlTable(Table table) {
- return new GenerateSequenceTable(table);
- }
+@Override
+public BeamSqlTable buildBeamSqlTable(Table table) {
+return new GenerateSequenceTable(table);
+}
Review comment:
Can we just set up the website to use the new code snippet widget with
```? Unless we add some explicit check to forbid ```, people will continue to
use that syntax since it's very common markdown syntax.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
