TheNeuralBit commented on a change in pull request #13565:
URL: https://github.com/apache/beam/pull/13565#discussion_r568261897
##########
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:
I don't think we should be making changes to a blog post's contents. It
looks like you've made changes to a lot of blog posts to remove leading
whitespace in code snippets.
Are these changes something you did while you were experimenting? Can they
be reverted?
----------------------------------------------------------------
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]
