text: I can see now why it extends into the river to "connect" to the line for the river, but is this appropriate
  The OSM wiki says "In order to allow routing, do ensure that the ends of the way connect correctly with and other related waterway features, including other canals, rivers, streams, lakes etc.", however there is no direct navigable connection between the river and the canal.