Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

flyway info does cuts off or truncates information #742

Closed
derylspielman opened this issue Apr 29, 2014 · 2 comments
Closed

flyway info does cuts off or truncates information #742

derylspielman opened this issue Apr 29, 2014 · 2 comments
Labels
Milestone

Comments

@derylspielman
Copy link

@derylspielman derylspielman commented Apr 29, 2014

Version and description are cut off when I run mvn flyway:info. I receive the following:

+----------------+----------------------------+---------------------+---------+
| Version | Description | Installed on | State |
+----------------+----------------------------+---------------------+---------+
| 1.0 | ddl | | PreInit |
| 1.1 | reference data | 2014-04-26 15:01:37 | Success |
| 2014.04.24.14. | new fields client asset va | 2014-04-26 15:04:03 | Success |

The full version should be:

+----------------+----------------------------+---------------------+---------+
| Version | Description | Installed on | State |
+----------------+----------------------------+---------------------+---------+
| 1.0 | ddl | | PreInit |
| 1.1 | reference data | 2014-04-26 15:01:37 | Success |
| 2014.04.24.14.29.19 | new fields client asset value | 2014-04-26 15:04:03 | Success |

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Apr 29, 2014

This is correct. The Maven plugin output defaults to a width of 79 characters. The command-line detects the width of the console and adjusts accordingly.

In Maven this isn't possible. So we're left with three choices:

  • Leave things as is (79 chars)
  • Increase to some other fixed limit (ex. 119 chars) and resize columns accordingly
  • Make it fully dynamic. Resize columns based on content with no truncating

Which direction would you favor and why?

@derylspielman
Copy link
Author

@derylspielman derylspielman commented Apr 30, 2014

I favor dynamic because that would be the end all fix. Everyone has different formats for their versions ranging from major.minor.patch to full dates with separators like mine so we cannot assume a maximum length. 20 something characters is not enough for a description either as it may contain "adding table blah_blah" which is on the cusp of being truncated. The current display of info is not expanding to the unused space in my console so it seems wasteful to have to truncate and hide information. Thanks!

@axelfontaine axelfontaine added this to the Flyway 3.1 milestone May 2, 2014
@axelfontaine axelfontaine added t: bug and removed m: Maven labels Nov 24, 2014
@axelfontaine axelfontaine changed the title mvn flyway:info does cuts off or truncates information flyway info does cuts off or truncates information Nov 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants