Developer forum

Hi Imar,

What you are downloading is a CSV file and we are using ; to separate columns. I do not get the ; on my solution when I download a single column. How are you viewing the file?

BR Rasmus Sanggaard

Excel may be hiding it for you when you import the file as data and you happen to have the semi-colon as your default delimiter. However, the semi-colon is in the raw data:

So, many versions of Excel / Excel configurations or other editors will see it.

Thanks!

Imar

Hi, 

I can't reproduce extacly what you get, but I see that we have problem when the content is too large. And some issues when using 'top'. I have created a bug for it.  #26539

BR Rasmus Sanggaard

Thanks! Is there a timeline for the new Firehose? I have a bunch of other bugs and requests but want to make sure they are not already addressed in the updated Firehose.

Hi,

The new and improved Firehose is released. Keep in mind that it all depends on what ring you are on. It reached R1 today. 

BR Rasmus Sanggaard

That is sssssssssooooooooooooooooo much better :)

And I'm glad I have a horizontal scroll wheel on my mouse to help with that :P

Yes, the new Firehose is much better. Great job! Here's all the good stuff:

- I love the new auto-suggest
- Yay!! to pascal-case navigation in auto-suggest
- Ctl+Enter is great
- Love the single screen much better than the separate overlays with results
- Great to have confirmation dialog when modifying data
- The "Rows affected" after modifying data is helpful
- Glad to see history back!

But there are a few things that would make the implementation perfect:

- Automatically put focus on text area when clicking Firehose in the menu so I can start typing right away
- The Download button now downloads a zero-bytes file for small result sets (or so it seems; can't point out exactly when it fails). Running this:

  Select top 10 ProductNumber FROM EcomProducts

  gives me a zero-bytes file.
- SQL errors should ideally not be logged under Monitoring. I think this is user error more than anything else and doesn't need monitoring.
  I.e. Querying for this 

  Select TOP 10 ProductNumbe FROM EcomProducts

  reports this:

     025-12-03 11:58:31.5005: WITH BaseQuery AS (
          Select TOP 10 ProductNumbe FROM EcomProducts
      )
      SELECT *
      FROM BaseQuery
      ORDER BY 1
      OFFSET 0 ROWS
      FETCH NEXT 96 ROWS ONLY. System.Data.SqlClient.SqlException (0x80131904): Invalid column name 'ProductNumbe'.
         at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)

- Selecting a history item automatically executes the query. I don't think it should, especially not with DML statements modifying data

Thanks!

Imar

And one more thing: pleaase make column headers in the result selectable with the mouse and double click. It's pretty common that you want to filter on or select a specific column only from a previous result set. Double-clicking on the column header to copy it and paste in the editor is a common thing I do.

Imar

Hi guys,

I am super pleased that you are happy about the improvements that we (finally) made (I am too :)) for the Firehose. While I understand that we still have some improvements that are missing, let's not make this thread into an "All we want for the Firehose" thread and let’s stay on topic. I have noted everything down, but for future ideas and thoughts, let's keep the Forum for bugs and questions and GitHub for feature requests.

BR Rasmus Sanggaard

Fair enough; I just got carried away by all the good new stuff! :-) Will use Github for feature requests.

Imar